Skip to content

Update README Render Comparison section with current stats#11

Merged
GOODBOY008 merged 1 commit into
mainfrom
copilot/update-readme-render-comparison
May 5, 2026
Merged

Update README Render Comparison section with current stats#11
GOODBOY008 merged 1 commit into
mainfrom
copilot/update-readme-render-comparison

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

README's Render Comparison section was stale — diff percentages, label count, and E2E test count all drifted from reality.

Changes

  • Diff table: correct per-label percentages to match current diff_report.txt (fedex 5.92%→5.77%, ups 6.91%→5.74%, dhlpaket 2.18%→2.17%, usps 4.80%→4.05%)
  • Summary line: 79→83 labels; perfect 4→6, good 24→27, minor 40→39
  • E2E test count: 57→82 (reflects actual #[test] count in tests/e2e_golden.rs)

@GOODBOY008 GOODBOY008 marked this pull request as ready for review May 5, 2026 09:38
Copilot AI review requested due to automatic review settings May 5, 2026 09:38
@GOODBOY008 GOODBOY008 self-requested a review May 5, 2026 09:39
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refreshes the README’s render-comparison metrics so the public documentation matches the repository’s current diff report and golden-test suite.

Changes:

  • Updated the per-label diff percentages shown in the Render Comparison table.
  • Updated the summary totals for labels tested and diff-quality buckets.
  • Updated the documented golden-file E2E test count in the Testing section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
```

57 golden-file E2E tests compare rendered output pixel-by-pixel against reference PNGs from the Labelary reference renderer.
82 golden-file E2E tests compare rendered output pixel-by-pixel against reference PNGs from the Labelary reference renderer.
@GOODBOY008 GOODBOY008 merged commit 65042f1 into main May 5, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants