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
Rework SkewT tranform #1125
Rework SkewT tranform #1125
Conversation
FWIW I like this PR template seeing it in practice now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll let you deal with the Codacy issues as you see fit. The Travis 2.7 test failures should be fun as well...but this looks good to me overall.
71489ec
to
38d8618
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending CI
990fcd7
to
13eb1e2
Compare
This changes to apply the skewing last, which fixes some issues caused by working in normalized axes coordinates, like problems when the aspect ratio of the plot wasn't square. This likely paves the way for better handling of data aspect ratio for skewT now too. The key here is to use matplotlib's transform framework to listen for changes to the axes bbox, so that the transform can update to use the proper origin for the skewing.
13eb1e2
to
bdcad95
Compare
Codacy is out to lunch. I think this is ready provided everything passes on this rebase. |
Description Of Changes
This changes to apply the skewing last, which fixes some issues caused by working in normalized axes coordinates, like problems when the aspect ratio of the plot wasn't square. This likely paves the way for better handling of data aspect ratio for skewT now too. The key here is to use matplotlib's transform framework to listen for changes to the axes bbox, so that the transform can update to use the proper origin for the skewing.
Checklist