Skip to content

Commit 2c7ffb4

Browse files
timtreisclaude
andcommitted
Fix multi-panel visual test: add title="" to avoid layout shift
The multi-panel frameon=False test produced different title positions across Python versions due to bbox_inches="tight" cropping differently when axes frames are off. Adding title="" eliminates the layout-sensitive text — the behavioral multi-panel assertion already covers title handling. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c0bcc40 commit 2c7ffb4

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

-90.8 KB
Binary file not shown.

tests/pl/test_show.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,10 @@ def test_plot_frameon_false_single_panel(self, sdata_blobs: SpatialData):
3333
"""Visual test: frameon=False hides axes decorations on a single panel (regression for #204)."""
3434
sdata_blobs.pl.render_images(element="blobs_image").pl.show(frameon=False)
3535

36-
def test_visual_frameon_false_multi_panel(self, sdata_blobs: SpatialData):
36+
def test_plot_frameon_false_multi_panel(self, sdata_blobs: SpatialData):
3737
"""Visual test: frameon=False hides axes decorations on all panels (regression for #204)."""
3838
set_transformation(sdata_blobs["blobs_image"], Identity(), "second_cs")
39-
sdata_blobs.pl.render_images(element="blobs_image").pl.show(frameon=False)
40-
self.compare("Show_frameon_false_multi_panel", tolerance=20)
39+
sdata_blobs.pl.render_images(element="blobs_image").pl.show(frameon=False, title="")
4140

4241
def test_plot_no_decorations(self, sdata_blobs: SpatialData):
4342
"""Visual test: frameon=False + title='' produces just the plot content (regression for #204)."""

0 commit comments

Comments
 (0)