Skip to content

Commit fe3850a

Browse files
Fix README visual export sizing
1 parent b82aee7 commit fe3850a

4 files changed

Lines changed: 19 additions & 0 deletions

File tree

docs/images/context-efficiency.png

-11 KB
Loading
-10.8 KB
Loading
-14.1 KB
Loading

tests/test_benchmark_evidence.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
import json
2+
import struct
23
from copy import deepcopy
34
from pathlib import Path
5+
from xml.etree import ElementTree
46

57
import pytest
68

@@ -101,6 +103,23 @@ def test_readme_makes_agent_benefits_and_visual_evidence_scannable():
101103
assert (ROOT / "docs" / "images" / filename).is_file()
102104

103105

106+
def test_readme_visual_pngs_match_their_svg_canvas():
107+
"""README image exports must not carry hidden screenshot padding."""
108+
image_dir = ROOT / "docs" / "images"
109+
110+
for stem in (
111+
"engraphis-benefit-flow",
112+
"evidence-backed-agent-examples",
113+
"context-efficiency",
114+
):
115+
svg = ElementTree.parse(image_dir / f"{stem}.svg").getroot()
116+
expected = (int(svg.attrib["width"]), int(svg.attrib["height"]))
117+
png_header = (image_dir / f"{stem}.png").read_bytes()[:24]
118+
119+
assert png_header[:8] == b"\x89PNG\r\n\x1a\n"
120+
assert struct.unpack(">II", png_header[16:24]) == expected
121+
122+
104123
def test_example_visual_uses_the_checked_in_offline_fixture_results():
105124
"""The new examples must not drift away from the commands readers can run."""
106125
longdoc = ROOT / "eval" / "datasets" / "longdoc.jsonl"

0 commit comments

Comments
 (0)