Commitcb47f2d

authored

Merge pull request#11754 from jklymak/fix-update-spine-positions-before-getting-extents

FIX: update spine positions before get extents

2 parents3020c21 +5545050 commitcb47f2dCopy full SHA for cb47f2d

File tree

+16

-8

lines changed

+16

-8

lines changed

Lines changed: 0 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -145,14 +145,6 @@ def do_constrained_layout(fig, renderer, h_pad, w_pad,`
`145`	`145`
`146`	`146`	`'''`
`147`	`147`
`148`		`-try:`
`149`		`-iffig.canvas.toolbar._activein ('PAN','ZOOM'):`
`150`		`-# don't do constrained layout during zoom and pan.`
`151`		`-return`
`152`		`-exceptAttributeError:`
`153`		`-# not toolbar, or no _active attribute..`
`154`		`-pass`
`155`		`-`
`156`	`148`	`invTransFig=fig.transFigure.inverted().transform_bbox`
`157`	`149`
`158`	`150`	`# list of unique gridspecs that contain child axes:`

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -146,6 +146,12 @@ def get_patch_transform(self):`
`146`	`146`	`else:`
`147`	`147`	`returnsuper().get_patch_transform()`
`148`	`148`
	`149`	`+defget_window_extent(self,renderer=None):`
	`150`	`+# make sure the location is updated so that transforms etc are`
	`151`	`+# correct:`
	`152`	`+self._adjust_location()`
	`153`	`+returnsuper().get_window_extent(renderer=renderer)`
	`154`	`+`
`149`	`155`	`defget_path(self):`
`150`	`156`	`returnself._path`
`151`	`157`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -5782,3 +5782,13 @@ def test_zoom_inset():`
`5782`	`5782`	`[0.8425,0.907692]])`
`5783`	`5783`	`np.testing.assert_allclose(axin1.get_position().get_points(),`
`5784`	`5784`	`xx,rtol=1e-4)`
	`5785`	`+`
	`5786`	`+`
	`5787`	`+deftest_spines_properbbox_after_zoom():`
	`5788`	`+fig,ax=plt.subplots()`
	`5789`	`+bb=ax.spines['bottom'].get_window_extent(fig.canvas.get_renderer())`
	`5790`	`+# this is what zoom calls:`
	`5791`	`+ax._set_view_from_bbox((320,320,500,500),'in',`
	`5792`	`+None,False,False)`
	`5793`	`+bb2=ax.spines['bottom'].get_window_extent(fig.canvas.get_renderer())`
	`5794`	`+np.testing.assert_allclose(bb.get_points(),bb2.get_points(),rtol=1e-6)`

Comments

(0)