You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't think #901 introduced this -- it removed a check (which was case sensitive) and allowed it to pass through to another part of the library (which is also case sensitive). I'm not sure I consider this a bug -- most things in matplotlib are case sensitive.
The relevant code appears to be in Collection.set_edgecolor, which converts everything to lower case. Although in general I prefer case-sensitivity, there are exceptions, and this is one; it is just too hard to remember that something should be "None", and something else should be "none". So, as far as I can see, things are working in a reasonable fashion.
Activity
mdboom commentedon Jul 10, 2012
I don't think #901 introduced this -- it removed a check (which was case sensitive) and allowed it to pass through to another part of the library (which is also case sensitive). I'm not sure I consider this a bug -- most things in matplotlib are case sensitive.
efiring commentedon Jul 15, 2012
The relevant code appears to be in Collection.set_edgecolor, which converts everything to lower case. Although in general I prefer case-sensitivity, there are exceptions, and this is one; it is just too hard to remember that something should be "None", and something else should be "none". So, as far as I can see, things are working in a reasonable fashion.