Skip to content

DOC: improve pie chart example #34288

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

Closed
MarcoGorelli opened this issue May 21, 2020 · 6 comments · Fixed by #34296
Closed

DOC: improve pie chart example #34288

MarcoGorelli opened this issue May 21, 2020 · 6 comments · Fixed by #34296

Comments

@MarcoGorelli
Copy link
Member

Location of the documentation

https://pandas.pydata.org/pandas-docs/dev/reference/api/pandas.DataFrame.plot.pie.html

Note: You can check the latest versions of the docs on master here.

Documentation problem

This example doesn't look great:
image

The legend covers the labels, and the two pies are so close that "mercury" is hard to read.

Suggested fix for documentation

Make a more readable plot

@MarcoGorelli MarcoGorelli added Docs Needs Triage Issue that has not been reviewed by a pandas team member good first issue and removed Needs Triage Issue that has not been reviewed by a pandas team member labels May 21, 2020
@wicky1234444
Copy link
Contributor

Hi @MarcoGorelli , I am new to open source. Can i work on this?

@MarcoGorelli
Copy link
Member Author

@wicky1234444 sure, pull requests are welcome - see the contributing guide for how to get started, and feel free to ask for help

@wicky1234444
Copy link
Contributor

Thanks. I will got through the contributing guide and start on the issue.

@Akshatt
Copy link
Contributor

Akshatt commented May 21, 2020

Hi @MarcoGorelli, is the issue assigned already? If not can i work on this issue too? I'm new as well

@MarcoGorelli
Copy link
Member Author

Looks like @wicky1234444 is already working on it (if so, @wicky1234444 , please comment 'take' so the issue is assigned to you)

@wicky1234444
Copy link
Contributor

@MarcoGorelli iam working on it.

mroeschke pushed a commit that referenced this issue May 23, 2020
* DOC:updated pie plot legend overlap issue

* DOC: pie plot white space removal
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants