Skip to content
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

Change in zorder of streamplot between 1.3.1 and 1.4.0 #5369

Closed
QuLogic opened this issue Oct 31, 2015 · 4 comments
Closed

Change in zorder of streamplot between 1.3.1 and 1.4.0 #5369

QuLogic opened this issue Oct 31, 2015 · 4 comments
Assignees
Milestone

Comments

@QuLogic
Copy link
Member

QuLogic commented Oct 31, 2015

While upgrading Cartopy's test images (SciTools/cartopy#354), it seems that the zorder of the lines in stream plots has changed. Originally, the lines used the default zorder from LineCollection, but as a result of #2225, the lines use the zorder default set for the method. The default zorder from LineCollection is 2; the default for the method's argument is 1.

If you look carefully at the plot, you can see that the coastlines move from below to above the stream lines:
streamplot

Z order is tricky, being a float, so I'm not whether adjusting the default won't cause issues.

@QuLogic
Copy link
Member Author

QuLogic commented Nov 24, 2015

Ping? Really, it's a simple change, but the question is whether it should be changed here or not?

@tacaswell
Copy link
Member

👍 for bumping the method zorder default back to 2

@tacaswell tacaswell added this to the next major release (2.0) milestone Nov 24, 2015
@efiring
Copy link
Member

efiring commented Nov 24, 2015

I agree with @tacaswell.

@WeatherGod
Copy link
Member

Fixed by #5567

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants