Skip to content

Task #134: bleed-measurement calibration pass (30 real catalog images) - #80

Merged
WilfordGrimley merged 2 commits into
masterfrom
claude/bleed-calibration-134
Jul 18, 2026
Merged

Task #134: bleed-measurement calibration pass (30 real catalog images)#80
WilfordGrimley merged 2 commits into
masterfrom
claude/bleed-calibration-134

Conversation

@WilfordGrimley

Copy link
Copy Markdown

Summary

  • Ran the shipped bleedNormalize.ts measurement algorithm against 30 real catalog images (fetched through the production image-CDN full-tier endpoint, respecting its shared rate limiter).
  • Ground truth assigned via the same aspect-ratio method the backend's already-validated classify_bleed_edge uses (28 bleed / 2 trimmed / 0 abstain).
  • Found a real, reproducible measurement bias (real bled cards measure ~2x their true bleed depth) and root-caused it (bleed+border color conflation, not a threshold or DPI issue - confirmed via a threshold sweep and dimension check).
  • All 4 named constants left at their defaults: no single one cleanly fixes this without an unreviewed resolveBleedPlan behavior change, so it's documented and tracked as a design follow-up instead.
  • Full data + reasoning: docs/reports/2026-07-18-bleed-calibration-134.md, bleedNormalize.ts's calibration comment, docs/proposals/proposal-b-bleed-normalization.md's "Tracked, not building" section.

Test plan

  • npx jest src/features/pdf/bleedNormalize.test.ts - 8/8 pass, unchanged (comment/docs-only diff, no functional code change)
  • npx prettier@2.7.1 --check on all touched files

Generated with Claude Code

… images

Constants unchanged - real bias found (bleed+border color conflation,
~2x overshoot) but no single constant fixes it without an unreviewed
resolveBleedPlan behavior change. Documented in bleedNormalize.ts's
calibration comment + docs/reports/2026-07-18-bleed-calibration-134.md,
tracked as a design follow-up in the proposal doc.
@WilfordGrimley
WilfordGrimley merged commit 8208326 into master Jul 18, 2026
7 checks passed
@WilfordGrimley
WilfordGrimley deleted the claude/bleed-calibration-134 branch July 18, 2026 18:36
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.

1 participant