[#90] detail=high bypasses the fmd floor — fine-grid trace for >1280 inputs#92
Merged
Merged
Conversation
…n the fine grid Root cause of the mustache faceting (operator real-usage): at detail=high a 1648x2944 flat portrait was downscaled to 1280 before tracing, and dark-on-dark region boundaries (mustache vs beard stripes) warp at that resolution — the curve distortion is a trace-resolution artifact, not a smooth-gate defect (verified: identical settings at cap 2048 restore the silhouette; PV traces the same input at native res). The input's fmd (7.84) sits just under the 8.0 relax floor, so #86 could not catch it. Fix: --detail high is an explicit max-fidelity opt-in — relax the cap to 2048 for any >1280 input without consulting the density detector (consistent with high's existing semantics: cov_cap 4000->14000 etc). Other levels keep the density gate. Explicit --max-input-edge-px still wins. Evidence: z-design .../analysis-2026-07-07/T90-mustache-orig-1280-2048-pv.png (orig | high@1280 warped | high@2048 restored | PV) + T90-hipster-high-cap2048.svg. ruff + 133 tests green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #90.
Root cause (measured, not the original hypothesis)
The mustache curve faceting at
detail=highis a trace-resolution artifact, not a smooth-gate defect: the 1648×2944 input was downscaled to 1280 before tracing, and dark-on-dark boundaries (mustache vs beard stripes) warp at that grid. Same settings at cap 2048 → silhouette restored (PV traces the same input at native res, 182p). The input's fmd = 7.84, just under the 8.0 relax floor, so #86's density gate couldn't catch it.Fix (minimal)
--detail high= explicit max-fidelity opt-in → always relax the cap to 2048 for >1280 inputs, no density gamble. Matches high's existing semantics (cov_cap 4000→14000, flatten-off). Normal/clean/poster keep the density gate; explicit--max-input-edge-pxstill wins; ≤1280 inputs untouched.Evidence
z-design/SVGSmith_Convert/new-test/analysis-2026-07-07/T90-mustache-orig-1280-2048-pv.png— orig | high@1280 (warped) | high@2048 (restored) | PV. SVG:T90-hipster-high-cap2048.svg(203p, 5.3s).Verify
ruff clean · 133 tests green (new: high relaxes >1280 flat, normal keeps 1280) · hipster live report note:
detail=high, cap relaxed 1280->2048.Release note
Should fold into 0.9.0 (PR #91) so the host gets #86+#90 together. Operator eye-gate: the montage above.
🤖 Generated with Claude Code