Conversation
lib/matplotlib/text.py
Outdated
There was a problem hiding this comment.
pep8 is choking on this line.
There was a problem hiding this comment.
True. I have modified the line to follow the PEP8 conventions.
-Michiel.
On Wed, 6/17/15, Thomas A Caswell notifications@github.com wrote:
Subject: Re: [matplotlib] Fix for issue4024 (#4532)
To: "matplotlib/matplotlib" matplotlib@noreply.github.com
Cc: "mdehoon" mjldehoon@yahoo.com
Date: Wednesday, June 17, 2015, 1:07 PM
In lib/matplotlib/text.py:
@@ -771,19 +774,23 @@ def
draw(self, renderer):
clean_line, ismath =
textobj.is_math_text(line)if textobj.get_path_effects():
from matplotlib.patheffects import- textrenderer = PathEffectRenderer(
textobj.get_path_effects(), renderer)
path_effects =if textobj.get_usetex():ismath = "TeX"path, transform =
angle, ismath)
pep8 is choking on this line.
—
Reply to this email directly or view
it on GitHub.
|
I have modified the offending line to follow the PEP8 conventions. |
|
failure looks like a transient, kicked to restart On Wed, Jun 17, 2015 at 10:37 AM mdehoon notifications@github.com wrote:
|
|
I am still not sure I fully follow what is going on here, but isn't this treating the symptom and not the disease (which is |
|
See discussion at #4202 I am strongly 👎 on merging this as it working around a bug in some of the backends which is affecting the code paths of the rest of the backends in a way that makes the code more complex. |
|
I'm closing this based on the discussion above and in #4202. |
3 participants
Fixes issue #4024 both for the MacOSX and the Cairo backends.