Skip to content

docs(benchmark): fix v0.19.0 chart bottom spacing (tick gap + divider) - #1297

Merged
ajianaz merged 1 commit into
developfrom
docs/chart-v019-bottom-spacing
Sep 29, 2026
Merged

ajianaz merged 1 commit into
developfrom
docs/chart-v019-bottom-spacing

Conversation

@ajianaz

@ajianaz ajianaz commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

What

Regenerates docs/assets/longmemeval-recall-v019.png with corrected bottom spacing: the 0-100% axis tick row moves from flush-against-the-last-bar to a ~36px gap below it (matching the approved v0.17.0 chart's rhythm), a subtle hairline divider is added between the chart body and the footer block, and the footer lines sit lower in the canvas. No text or data changes: same headline (98.2%), same badge, same footer copy.

Why

Owner review feedback on the merged chart (2026-09-29): the tick row hugged the agentmemory bar (0-5px gap) and the card lacked a visible separation before the footer. Pixel-verified on the regenerated render: bar bottom y524, ticks y563-577, divider (rgb 48,48,62) at y=594, footer y~606-670, all within the 720px canvas.

Testing

Owner review feedback: tick row hugged the last bar; add hairline divider above footer. Data unchanged.
@github-actions

Copy link
Copy Markdown

🔍 Cora AI Code Review

✅ No issues found. Code looks good!


Review powered by cora-code · BYOK · MIT

@ajianaz
ajianaz merged commit 4a7606c into develop Sep 29, 2026
15 checks passed
@ajianaz
ajianaz deleted the docs/chart-v019-bottom-spacing branch September 29, 2026 09:28
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