| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
This also raises a question: does anyone know why scatter still has the undocumented "verts" kwarg? We should either remove it, with deprecation if necessary, or include it in the docstring. |
Sorry, something went wrong.
|
I am fine with removing it as it looks like it is one more way to specify the marker shape. Probably do need deprecation cycle though. marker can also be array like as MarkerStyle can take arrays as input. |
Sorry, something went wrong.
I also tried to clean up some of the scatter code and documentation, together with related documentation in collections. I added a test.
|
Wow. Good find guys. |
Sorry, something went wrong.
Issue fixed on matplotlib master matplotlib/matplotlib#4079
There was a problem hiding this comment.
Coming in late to the discussion but I have just spent a few minutes trying to determine what I did wrong on my scatter graphs...
I thought c stood as an acronym for color as fc stands for facecolor etcetera... however this does not apply here, specifying color='green' instead of c='green' means you set the edge-colour to green instead of the face-colour...
Should I open an issue/PR about this? Or can people live with this inconsistency?
Sorry, something went wrong.
There was a problem hiding this comment.
As far as I understand it, 'c' should be an alias for 'color'. Furthermore,
'color' should specify both edge and face colors, unless explicitly
overriden by 'facecolor' and such.
On Wed, Apr 29, 2015 at 12:45 PM, OceanWolf notifications@github.com
wrote:
In lib/matplotlib/axes/_axes.py
#4079 (comment):
if ec is not None: edgecolors = ecfc = kwargs.pop('facecolor', None) if fc is not None: facecolors = fcfc = kwargs.pop('facecolors', None) if fc is not None: facecolors = fc # 'color' should be deprecated in scatter, or clearly defined; # since it isn't, I am giving it low priority.co = kwargs.pop('color', None) if co is not None: if edgecolors is None: edgecolors = co if facecolors is None: facecolors = coComing in late to the discussion but I have just spent a few minutes
trying to determine what I did wrong on my scatter graphs...I thought c stood as an acronym for color as fc stands for facecolor
etcetera... however this does not apply here, specifying color='green'
instead of c='green' means you set the edge-colour to green instead of
the face-colour...Should I open an issue/PR about this? Or can people live with this
inconsistency?—
Reply to this email directly or view it on GitHub
https://github.com/matplotlib/matplotlib/pull/4079/files#r29355046.
Sorry, something went wrong.
There was a problem hiding this comment.
This is a confusing mess for historical reasons. It looks like there might be a need for "facecolors=None" in the arg list, along with "edgecolors=None". I suspect I had a reason for not including this in the PR, but I don't remember; maybe it is worth a try.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
I also tried to clean up some of the scatter code and documentation,
together with related documentation in collections. I added a test.