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
PEP8 fixes on textpath.py #1375
Conversation
@@ -478,13 +484,13 @@ def _revalidate_path(self): | |||
|
|||
""" | |||
if self._invalid or \ |
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.
Trailing backslash.
On this file, I also have those warnings: textpath.py:253: W806 local variable 'glyph' is assigned to but never used |
@@ -113,7 +113,8 @@ def get_text_width_height_descent(self, s, prop, ismath): | |||
|
|||
def get_text_path(self, prop, s, ismath=False, usetex=False): | |||
""" | |||
convert text *s* to path (a tuple of vertices and codes for matplotlib.math.Path). | |||
convert text *s* to path (a tuple of vertices and codes for | |||
matplotlib.math.Path). |
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.
Can we slip in a quick fix to this docstring: 'matplotlib.math.Path' --> 'matplotlib.path.Path'
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.
Good catch :)
@NelleV Perhaps it would make sense to consolidate all of the imports at the top of the file, and organize them with standard library first, then external libraries (numpy), then mpl stuff. |
It's done. |
PEP8 fixes on the textpath module.
Thanks,
N