-
-
Notifications
You must be signed in to change notification settings - Fork 144
ENH: Improve plotting types #278
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
Add types for many inputs Clean out non-public functions
Add tests Correct types
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.
Might be good in the tests to have one test where we test all the kwargs
documented in DataFrame.plot()
Add covered types in docs Add tests
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 think all of the keywords in DataFrame.plot()
can also be passed to things like DataFrame.plot.line()
, etc. So I think we should copy the keywords you included in __call__()
over to each of the individual functions. E.g., the following is valid:
foo = pd.DataFrame({"x":[i for i in range(10)], "y":[i*i for i in range(10)]})
foo.plot.line("x", "y", figsize=(3,3))
You don't have to test each one.
I'm hesitant to do this since not all keywords are accepted by all plot types and some, even if accepted, have no effect. For example, |
Fair point. It's not clear to me if people would do |
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.
thanks @bashtage
Add types for many inputs
Clean out non-public functions
assert_type()
to assert the type of any return value