-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: Allow plotting backend to be an option #28622
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
Conversation
Hmm I haven't added any docs. I suppose there should at least be a mention in the docstring. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add backend
to the docstring (and possibly the signature) for all of the plotting methods? That'll make this a bit more discoverable.
I think this is ready. |
d241e09
to
2199210
Compare
@jsignell can you rebase |
Co-Authored-By: Tom Augspurger <[email protected]>
2199210
to
0a7800b
Compare
pandas/plotting/_core.py
Outdated
... return_type=None) | ||
>>> type(boxplot) | ||
<class 'numpy.ndarray'> | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like not to duplicate this, but I couldn't figure out a good way to inject the backend argument doc string.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to clarify: this is because boxplot
doesn't accept a backend
argument right?
I may have a fix, will push it up if that works.
|
||
If ``return_type`` is `None`, a NumPy array | ||
of axes with the same shape as ``layout`` is returned. | ||
%(backend)s\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only really new thing. Otherwise it's just a move from being written in boxplot
to using Appender.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks @jsignell
Thanks @jsignell! |
* Allow backend to be an option
* Allow backend to be an option
* Allow backend to be an option
.plot()
#28619black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff