Skip to content

Visual evidence: record #1447 line-cap defect - #1449

Closed
developer0hye wants to merge 1 commit into
mainfrom
evidence/issue-1447-line-cap
Closed

Visual evidence: record #1447 line-cap defect#1449
developer0hye wants to merge 1 commit into
mainfrom
evidence/issue-1447-line-cap

Conversation

@developer0hye

Copy link
Copy Markdown
Owner

Summary

Related issue

Documents #1447

Testing

  • inspected the full page 8 GT/output renders, the 5% fuzz diff, and the matched title/rule crop at full resolution
  • verified the source open custom-geometry line has no cap attribute and therefore uses DrawingML's flat default
  • verified compare.jpg is 3000x350 with equal 1500px panels, progressive JPEG quality 86, stripped metadata, and 300 DPI JFIF density
  • python3 scripts/check_visual_pr.py evidence validation logic — passed for assets/bugfixes/issue-1447/compare.jpg
  • mandatory documentation freshness review: PASS

Visual impact

  • No rendered PDF change
  • Rendered PDF change or visual evidence added
  • Reason: Adds defect evidence for the open line-cap issue; rendering code is unchanged.

Visual audit

Visual comparison

Compare
Compare

Required inspection

  • Rendered all evidence at 150 DPI or higher
  • Stored progressive JPEG quality 86 assets with metadata stripped
  • Used Codex/Claude vision to inspect the full GT/output pages, diff, and matched crops
  • Inspected matched region crops at full resolution
  • Ran compare_layout.py --audit and dispositioned every large text-instance shift and painted-visibility mismatch
  • Ran the 5% fuzz pixel-difference sweep
  • Inventoried hairlines and border dash styles
  • Inventoried font weight, italic, and underline emphasis

Deviation audit

Check Result
Page count/order Matches GT — page 8 is compared in source order
Element presence Matches GT — the slide title and its rule are present in both panels
Position/size Remaining: #1435 and #1436 track legend placement, #1437 tracks the plot rectangle, and #1439 tracks legend-key size
Rotation/flip Matches GT — the title and rule have no differing rotation or flip
Fill Matches GT — the title and rule colors match; other page fills remain present
Stroke/border Remaining: #1447 tracks the rounded title-rule caps; its thickness, length, color, and solid dash style otherwise match
Shape outline geometry Remaining: #1447 tracks the open rule's endpoint geometry and #1439 tracks legend-key square dimensions
Text content Matches GT — visible page-8 text is unchanged
Font family/weight/style Remaining: #1424 tracks the chart-title styling; the slide-title emphasis matches GT
Text color Remaining: #1424 tracks the chart-title color; the slide-title color matches GT
Alignment Remaining: #1435 and #1436 track legend alignment; the slide title and rule align with GT
Line/paragraph spacing Matches GT — the title line is unchanged and the legend pitch fix from #1434 remains present
Clipping/overflow No deviation observed — neither the title nor the rule is clipped

Checklist

  • Commits include a Signed-off-by line
  • PR scope contains one root cause
  • Remaining visual deviations each reference an open issue

Signed-off-by: Yonghye Kwon <developer.0hye@gmail.com>
@developer0hye

Copy link
Copy Markdown
Owner Author

Closing this evidence-only PR before merge because the #1407 provenance and initial line-cap root-cause claim were incorrect. Issue #1447 remains open with the correction and the evidence branch is retained.

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