Commit166e9cc

committed

Merge pull request#4250 from efiring/quiver_copy

BUG : Quiver must copy U, V, C args so they can't change before draw()

2 parents9cf4d3e +5a88c5e commit166e9ccCopy full SHA for 166e9cc

File tree

+14

-3

lines changed

+14

-3

lines changed

Lines changed: 5 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -529,11 +529,13 @@ def draw(self, renderer):`
`529`	`529`	`mcollections.PolyCollection.draw(self,renderer)`
`530`	`530`
`531`	`531`	`defset_UVC(self,U,V,C=None):`
`532`		`-U=ma.masked_invalid(U,copy=False).ravel()`
`533`		`-V=ma.masked_invalid(V,copy=False).ravel()`
	`532`	`+# We need to ensure we have a copy, not a reference`
	`533`	`+# to an array that might change before draw().`
	`534`	`+U=ma.masked_invalid(U,copy=True).ravel()`
	`535`	`+V=ma.masked_invalid(V,copy=True).ravel()`
`534`	`536`	`mask=ma.mask_or(U.mask,V.mask,copy=False,shrink=True)`
`535`	`537`	`ifCisnotNone:`
`536`		`-C=ma.masked_invalid(C,copy=False).ravel()`
	`538`	`+C=ma.masked_invalid(C,copy=True).ravel()`
`537`	`539`	`mask=ma.mask_or(mask,C.mask,copy=False,shrink=True)`
`538`	`540`	`ifmaskisma.nomask:`
`539`	`541`	`C=C.filled()`

Lines changed: 9 additions & 0 deletions

Comments

(0)