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

Non-default color_cycle not working in Pie plot #4975

Closed
rubennj opened this issue Aug 21, 2015 · 5 comments
Closed

Non-default color_cycle not working in Pie plot #4975

rubennj opened this issue Aug 21, 2015 · 5 comments
Milestone

Comments

@rubennj
Copy link

rubennj commented Aug 21, 2015

Changing the default color_cycle works in other charts but not in pie.

import matplotlib as mpl
import matplotlib.pylab as plt

mpl.rcParams['axes.color_cycle'] = ['c', 'm', 'y', 'k']

fig, ax = plt.subplots()

ax.pie([15, 30, 45, 10])

New colors can be used only changing the colors parameter of pie.

Versions: MPL 1.4.3 - Python 3.4.1 - Windows

@WeatherGod
Copy link
Member

Yes, the "color cycle" in the pie() function was a quick-n-dirty kludge,
IIRC. The function hard-codes the cycle and doesn't use the internal cycle.
It shouldn't be too difficult to get it hooked up to the new property
cycling feature I am working on right now.

On Fri, Aug 21, 2015 at 6:15 AM, Ruben Nuñez notifications@github.com
wrote:

Changing the default color_cycle works in other charts but not in pie.

import matplotlib as mpl
import matplotlib.pylab as plt

mpl.rcParams['axes.color_cycle'] = ['c', 'm', 'y', 'k']

fig, ax = plt.subplots()

ax.pie([15, 30, 45, 10])

New colors can be used only changing the colors parameter of pie.

Versions: MPL 1.4.3 - Python 3.4.1 - Windows


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

@tacaswell tacaswell added this to the proposed next point release milestone Aug 21, 2015
@languitar
Copy link
Contributor

👍

@mdboom mdboom modified the milestones: next major release (2.0), proposed next point release (2.1) Nov 23, 2015
@mdboom
Copy link
Member

mdboom commented Nov 23, 2015

I'm remilestoning this as 2.0. Since we're changing the default color cycle for 2.0, we should definitely fix the hard-coded one in pie.

@mdboom
Copy link
Member

mdboom commented Jan 25, 2016

This is fixed by #5674.

@tacaswell
Copy link
Member

closed by #6291

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

5 participants