Skip to content

examples/api/skewt.py is not displaying the right part of the grid by default #6873

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.

Sign up for GitHub

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

Jump to bottom
Closed
anntzer opened this issue Jul 31, 2016 · 1 comment
Closed

examples/api/skewt.py is not displaying the right part of the grid by default #6873

anntzer opened this issue Jul 31, 2016 · 1 comment
Assignees
@dopplershift
Milestone
v2.0.0

Comments

@anntzer
Copy link
Contributor

anntzer commented Jul 31, 2016

mpl2.0b3/master Qt5Agg numpy1.11.1 Python 3.5.1

skewt.py is missing the left part of the grid by default:
skewt
Triggering a redraw (eg by panning the axes) solves the issue, but it may be something deeper?
@dopplershift dopplershift added this to the 2.0 (style change major release) milestone Nov 17, 2016
@dopplershift dopplershift self-assigned this Nov 17, 2016
@dopplershift
Copy link
Contributor

This was relying upon the upper spine's _adjust_location() method to update an interval on the xaxis for that upper range (ewww....); for some reason it wasn't getting called on that initial draw, but I'm not convinced that's a bug. My original design for handling that update was just....so bad and brittle, violating so many things, it needed to be redone.

I've got a cleaned up version working now that makes a lot more sense.

@dopplershift dopplershift mentioned this issue Nov 17, 2016
Merged
dopplershift added a commit to dopplershift/MetPy that referenced this issue Nov 18, 2016
@dopplershift
MNT: Refactor Skew-T support a bit.
2fb18b7 
Matches the work done in matplotlib/matplotlib#7479, which fixes missing
gridlines identified in matplotlib/matplotlib#6873. This is a cleaner
implementation of how this is handled anyways.
dopplershift added a commit to dopplershift/MetPy that referenced this issue Nov 18, 2016
@dopplershift
MNT: Refactor Skew-T support a bit.
cadea96 
Matches the work done in matplotlib/matplotlib#7479, which fixes missing
gridlines identified in matplotlib/matplotlib#6873. This is a cleaner
implementation of how this is handled anyways.
dopplershift added a commit to Unidata/MetPy that referenced this issue Nov 18, 2016
@dopplershift
MNT: Refactor Skew-T support a bit.
e13949c 
Matches the work done in matplotlib/matplotlib#7479, which fixes missing
gridlines identified in matplotlib/matplotlib#6873. This is a cleaner
implementation of how this is handled anyways.
ShawnMurd pushed a commit to ShawnMurd/MetPy that referenced this issue Dec 1, 2016
@dopplershift
MNT: Refactor Skew-T support a bit.
e63f763 
Matches the work done in matplotlib/matplotlib#7479, which fixes missing
gridlines identified in matplotlib/matplotlib#6873. This is a cleaner
implementation of how this is handled anyways.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dopplershift dopplershift

Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

No branches or pull requests
2 participants
@dopplershift @anntzer