Skip to content
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

Scatter markers do not follow new colour cycle #5990

Closed
QuLogic opened this issue Feb 12, 2016 · 3 comments
Closed

Scatter markers do not follow new colour cycle #5990

QuLogic opened this issue Feb 12, 2016 · 3 comments
Assignees
Milestone

Comments

@QuLogic
Copy link
Member

QuLogic commented Feb 12, 2016

See scatter_profile or scatter_hist.

If I understood the code correctly, it should be using patch.facecolor, but somewhere along the line that doesn't get applied.

@QuLogic QuLogic added this to the 2.0 (style change major release) milestone Feb 12, 2016
@WeatherGod
Copy link
Member

scatter has never used the color cycle. Do you mean "the new color scheme"?

On Fri, Feb 12, 2016 at 3:20 AM, Elliott Sales de Andrade <
notifications@github.com> wrote:

See scatter_profile
http://matplotlib.org/devdocs/examples/pylab_examples/scatter_profile.html
or scatter_hist
http://matplotlib.org/devdocs/examples/axes_grid1/scatter_hist.html.

If I understood the code correctly, it should be using patch.facecolor,
but somewhere along the line that doesn't get applied.


Reply to this email directly or view it on GitHub
#5990.

@tacaswell
Copy link
Member

There is probably a hard-coded-to-blue someplace in scatter. Even if it does not respect the color cycle, it should at least use it's first color as the un-mapped color.

@mdboom
Copy link
Member

mdboom commented Mar 7, 2016

I think the best fix here is to implement the "numbered color syntax" as proposed in #5674 and then change the hardcoded 'b' in scatter to '[0]'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants