8000 Fixes #5522, bug in custom scale example by jeffrey-hokanson · Pull Request #5533 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Fixes #5522, bug in custom scale example #5533

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

Merged
merged 2 commits into from
Nov 20, 2015

Conversation

jeffrey-hokanson
Copy link
Contributor

Added the property has_inverse = True to the custom scale example so that if axhline is later called on this axis an error is not produced.

Added the property has_inverse = True to the custom scale example
so that if axhline is later called on this axis an error is not
produced.
@@ -103,7 +103,7 @@ class MercatorLatitudeTransform(mtransforms.Transform):
input_dims = 1
output_dims = 1
is_separable = True

has_inverse = True
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a PEP8 nitpick here: Insert a blank space before the method definition.

@mdboom
Copy link
Member
mdboom commented Nov 20, 2015

Other than the PEP8 failure, looks good to me.

@mdboom mdboom added this to the Critical bugfix release (1.5.1) milestone Nov 20, 2015
@@ -138,6 +138,7 @@ class InvertedMercatorLatitudeTransform(mtransforms.Transform):
input_dims = 1
output_dims = 1
is_separable = True
has_inverse = True

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've fixed this and the file appears clear upon running pep8. Is there something else I'm missing?

tacaswell added a commit that referenced this pull request Nov 20, 2015
Fixes #5522, bug in custom scale example
@tacaswell tacaswell merged commit 7f46613 into matplotlib:master Nov 20, 2015
@tacaswell
Copy link
Member

@jeffrey-hokanson Thanks! Congratulations on (what I think) is your first contribution to mpl!

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

Successfully merging this pull request may close these issues.

3 participants
0