-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
QuadMesh.draw(): convert_xunits broken #560
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Thanks @bfroehle: can you provide an example where this is manifest? |
This is a blatant bug; the code as it still exists makes no sense whatsoever. The only question is whether to just put in the commas, or whether the whole block of code involving offsets can be deleted. Are offsets supported in QuadMesh? |
Answering my own question, yes, it looks like offsets are supported, so I added the commas. |
bug: fix unit support for offsets in QuadMesh; closes #560
bug: fix unit support for offsets in QuadMesh; closes #560
In QuadMesh.draw(), I think the following code is broken
xs = self.convert_xunits(self._offsets[:0]) ys = self.convert_yunits(self._offsets[:1])
because it is missing a comma between the : and 0 (or : and 1).
The text was updated successfully, but these errors were encountered: