Commit80b7c57

committed

MNT: move creating interal ScalarMappable to where it is used

This make it clearer that we only need this in one of the code pathsand avoids creating the un-used ScalarMappable in the png savefig codepath.

1 parentd66f959 commit80b7c57Copy full SHA for 80b7c57

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1553,8 +1553,6 @@ def imsave(fname, arr, vmin=None, vmax=None, cmap=None, format=None,`
`1553`	`1553`	`else:`
`1554`	`1554`	`# Don't bother creating an image; this avoids rounding errors on the`
`1555`	`1555`	`# size when dividing and then multiplying by dpi.`
`1556`		`-sm=cm.ScalarMappable(cmap=cmap)`
`1557`		`-sm.set_clim(vmin,vmax)`
`1558`	`1556`	`iforiginisNone:`
`1559`	`1557`	`origin=mpl.rcParams["image.origin"]`
`1560`	`1558`	`iforigin=="lower":`
`@@ -1567,6 +1565,8 @@ def imsave(fname, arr, vmin=None, vmax=None, cmap=None, format=None,`
`1567`	`1565`	`# as is, saving a few operations.`
`1568`	`1566`	`rgba=arr`
`1569`	`1567`	`else:`
	`1568`	`+sm=cm.ScalarMappable(cmap=cmap)`
	`1569`	`+sm.set_clim(vmin,vmax)`
`1570`	`1570`	`rgba=sm.to_rgba(arr,bytes=True)`
`1571`	`1571`	`ifpil_kwargsisNone:`
`1572`	`1572`	`pil_kwargs= {}`

Comments

(0)