DOC:updated pie plot legend overlap issue, closes #34288 #34296
Conversation
There was a problem hiding this comment.
Thanks @wicky1234444 , this looks better
Did you manage to build this page of the documentation locally?
Also, please put the issue number in the description (closes #<issue number goes here>)
pandas/plotting/_core.py
Outdated
| @@ -1475,7 +1475,7 @@ def pie(self, **kwargs): | |||
| .. plot:: | |||
| :context: close-figs | |||
|
|
|||
| >>> plot = df.plot.pie(subplots=True, figsize=(6, 3)) | |||
| >>> plot = df.plot.pie(subplots=True, figsize=(11, 11)) | |||
There was a problem hiding this comment.
Like this there's lots of whitespace above and below, perhaps do this?
| >>> plot = df.plot.pie(subplots=True, figsize=(11, 11)) | |
| >>> plot = df.plot.pie(subplots=True, figsize=(11, 5)) |
There was a problem hiding this comment.
yeah. I dropping a link to the image here.
There was a problem hiding this comment.
ok, i will change that.
There was a problem hiding this comment.
sorry i forgot add this in the description . closes #34288
There was a problem hiding this comment.
i have added the suggested changes. closes #34288
There was a problem hiding this comment.
Great, thanks!
closes #34288
Could you add this to the pull request description? I don't know if it works as a keyword if it's just in a comment
wicky1234444
changed the title
|
Thanks @wicky1234444 |
black pandasgit diff upstream/master -u -- "*.py" | flake8 --diff