-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
line color='none' regression in 1.3 #2760
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
Comments
Looking at this very quickly (and trying to remember what I did a year ago), it looks like I copied the code a few lines above I think the fix is to add |
added failing test for issue matplotlib#2760
Fixes matplotlib#2760 Fixes miss-handling of color conversion logic introduced in edc48f0
Just a caution, without inspecting the code in question: the general rule in mpl is that |
I had the same thought as @efiring here. I think the path of least change here is (which restores the use of
|
Your suggestion would make rgba_ln indexable, and would silently work in subsequent code that indexes what it thinks is a 3 or 4 element tuple. |
I started down that path last night as well, but it generates a bunch of duplicate code (both in LoC and at run time) as the logic for turning I am wary of changing the behavior in |
I'm trying to confirm, but when I import matplotlib after re-installing it, I get
It's probably my fault, but I don't have time to dig into this right now. If the test script above works for you, then it probably solves my problem. |
Hi,
The following code works in version 1.2, but leads to an exception in version 1.3:
I think the issue boils down to this commit: edc48f0
I'm not sure whether this was an intentional change. Thanks!
The text was updated successfully, but these errors were encountered: