Skip to content

Pie demo crashes backport v2.2.2-doc #11103

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.

Sign up for GitHub

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

Jump to bottom
Closed
jklymak opened this issue Apr 22, 2018 · 2 comments
Closed

Pie demo crashes backport v2.2.2-doc #11103

jklymak opened this issue Apr 22, 2018 · 2 comments
Labels
Documentation
Milestone
v2.2-doc

Comments

@jklymak
Copy link
Member

jklymak commented Apr 22, 2018 •
edited
Loading

Bug report

From #11083 and its back port #11089; see backport #11102 for the new error.

Warning, treated as error:
/home/circleci/project/examples/pie_and_polar_charts/pie_demo2.py failed to execute correctly: Traceback (most recent call last):
  File "/home/circleci/project/examples/pie_and_polar_charts/pie_demo2.py", line 32, in <module>
    shadow=True, radius=0.5)
  File "/home/circleci/project/lib/matplotlib/__init__.py", line 1855, in inner
    return func(ax, *args, **kwargs)
  File "/home/circleci/project/lib/matplotlib/axes/_axes.py", line 2930, in pie
    **textprops)
TypeError: text() got multiple values for keyword argument 'size'

I'd try and fix, but I'm still not comfortable pushing on the back branches. This snuck in while the doc builds weren't working... ping @timhoffm and @dstansby
@jklymak jklymak added this to the v2.2-doc milestone Apr 22, 2018
@ImportanceOfBeingErnest
Copy link
Member

A smaller issue is that the new pie_demo2 is not using equal aspect in the assumption that this is done interally, which is not the case for version 2.2. A backported version of this should still set equal aspect manually in the example.

@dstansby dstansby mentioned this issue Apr 23, 2018
Merged
@dstansby
Copy link
Member

I reverted that change, so now I think this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
v2.2-doc
Development

No branches or pull requests
3 participants
@jklymak @dstansby @ImportanceOfBeingErnest