Commit7fcb063

authored

fix#29410 Modifying Axes' position also alters the original Bbox object used for initialization (#29411)

*fix#29410* Update test_axes_set_position_external_bbox_unchanged to use figure comparison

1 parentca8079b commit7fcb063Copy full SHA for 7fcb063

File tree

+12

-1

lines changed

+12

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -641,7 +641,7 @@ def __init__(self, fig,`
`641`	`641`	`args= (rect,)`
`642`	`642`	`subplotspec=None`
`643`	`643`	`iflen(args)==1andisinstance(args[0],mtransforms.Bbox):`
`644`		`-self._position=args[0]`
	`644`	`+self._position=args[0].frozen()`
`645`	`645`	`eliflen(args)==1andnp.iterable(args[0]):`
`646`	`646`	`self._position=mtransforms.Bbox.from_bounds(*args[0])`
`647`	`647`	`else:`

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -9610,3 +9610,14 @@ def test_bar_color_precedence():`
`9610`	`9610`	`bars=ax.bar([31,32,33], [4,5,6],color='red',facecolor='green')`
`9611`	`9611`	`forbarinbars:`
`9612`	`9612`	`assertmcolors.same_color(bar.get_facecolor(),'green')`
	`9613`	`+`
	`9614`	`+`
	`9615`	`+@check_figures_equal(extensions=['png'])`
	`9616`	`+deftest_axes_set_position_external_bbox_unchanged(fig_test,fig_ref):`
	`9617`	`+# From #29410: Modifying Axes' position also alters the original Bbox`
	`9618`	`+# object used for initialization`
	`9619`	`+bbox=mtransforms.Bbox([[0.0,0.0], [1.0,1.0]])`
	`9620`	`+ax_test=fig_test.add_axes(bbox)`
	`9621`	`+ax_test.set_position([0.25,0.25,0.5,0.5])`
	`9622`	`+assert (bbox.x0,bbox.y0,bbox.width,bbox.height)== (0.0,0.0,1.0,1.0)`
	`9623`	`+ax_ref=fig_ref.add_axes([0.25,0.25,0.5,0.5])`

Comments

(0)