-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Document get_{x,y}axis_transform more prominently. #9375
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
Conversation
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.
small typo
Sorry, something went wrong.
All reactions
=========== ============================= ===================================== | ||
Coordinates Transformation object Description | ||
=========== ============================= ===================================== | ||
"data" ``ax.transData```` The userland data coordinate system, |
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.
What is a "userland"?
Sorry, something went wrong.
All reactions
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.
"from the POV of the user" but I agree it's a bit obscure. "The data coordinate system" actually sounds good enough?
Sorry, something went wrong.
All reactions
-
👍 1 reaction
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.
force pushed (also fixed below)
Sorry, something went wrong.
All reactions
display, and (width, height) is top | ||
right of the display in pixels. | ||
"xaxis", ``ax.get_xaxis_transform()``, Blended coordinate systems; use data | ||
"yaxis" ``ax.get_yaxis_transform()`` coordinates one one of the axis and |
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.
one one -> on one
Sorry, something went wrong.
All reactions
938cbdc
to
8d8a3fc
Compare
They are quite useful but right now are documented in a slightly obscure manner.
8d8a3fc
to
f19eddb
Compare
... there I went nuts and merged it. Hope that was OK - docs just require one approval, right? |
All reactions
Sorry, something went wrong.
Document get_{x,y}axis_transform more prominently.
backported to v2.1.0-doc as d5a3e46 |
All reactions
Sorry, something went wrong.
jklymak
Successfully merging this pull request may close these issues.
2.1 - new problem with log ax.transData
They are quite useful but right now are documented in a slightly obscure
manner.
Closes #9369.
PR Summary
PR Checklist