-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: allow plot, boxplot, hist and completion on GroupBy objects #5105
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
BUG: allow plot, boxplot, hist and completion on GroupBy objects #5105
Conversation
should those plot tests be marked slow? |
@@ -2728,6 +2727,85 @@ def test_groupby_whitelist(self): | |||
with tm.assertRaisesRegexp(AttributeError, msg): | |||
getattr(gb, bl) | |||
|
|||
def test_series_groupby_plotting_nominally_works(self): |
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 test is pretty quick so not marking as @slow
good maybe put a tab complete example for say frames in docs as well (basics?) later if u can |
done ... i put in |
gre8...looks fine to me |
BUG: allow plot, boxplot, hist and completion on GroupBy objects
@nehalecky there you go :) |
Grand, thank you so much @cpcloud, this works perfectly! :) |
closes #5102
also fixes extra plots created by groupby plotting (couldn't find an issue), now there's a test that ensures that groupby only plots the number of axes/figures it's supposed to
removes an unused internal groupby parameter