Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

pie_chart_with_horizontal_bar_chart added#2578

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
ghost wants to merge1 commit intomatplotlib:masterfromunknown repository
Closed

pie_chart_with_horizontal_bar_chart added#2578

ghost wants to merge1 commit intomatplotlib:masterfromunknown repository

Conversation

ghost
Copy link

Simple Bar chart with leading horizontal bar chart added to examples/pie_and_polar_charts section.

@tacaswell
Copy link
Member

Can you also add this to the documentation someplace (gallery?)

Please seehttps://github.com/matplotlib/matplotlib/wiki/MEP12,#2181 and#2474

@tonysyu
Copy link
Contributor

I hope this doesn't come off the wrong way, but I don't really understand the point of this example. There are examples for bar charts and pie charts that demonstrate the functionality here, and I don't think combining the two into one plot warrants a new example.

Sorry to be so negative, but I'm of the opinion that the gallery needs to be trimmed down.

@efiring
Copy link
Member

I agree with@tonysu on this.

@ghost
Copy link
Author

The idea behind this example was to add more example to the Library. In my view more example we have the more useful the Library is.
@tonysyu is right with his point of view. The example does not add any new functionality but serve as an another use of this two charts types.

@ghost
Copy link
Author

After reviewing the MEP12 (which@tacaswell suggested) more carefully, not to add this new example to the Library may be more reasonable to keep example sections simple and not to repeat ourselves.
In my opinion, having more and more example is quite important for useful library. On the other hand, examples should be organized carefully for usability and simplicity. (@tonysyu is right in his point)

So, not to merge this patch will be the best.

@ghostghost closed thisNov 8, 2013
This pull request wasclosed.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@tacaswell@tonysyu@efiring

[8]ページ先頭

©2009-2025 Movatter.jp