-
Notifications
You must be signed in to change notification settings - Fork 891
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
Fixed documentation bug in groupby agg method #8325
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
ok to test |
Thanks for the fix, LGTM! |
@gpucibot merge |
I added another commit with similar non-functional documentation fix in the pipe method. Not sure, whether I should have added it to this pull request. |
That's perfectly fine, thanks for the fixes! |
rerun tests |
3 similar comments
rerun tests |
rerun tests |
rerun tests |
Rerun tests. |
Codecov Report
@@ Coverage Diff @@
## branch-21.08 #8325 +/- ##
===============================================
Coverage ? 82.83%
===============================================
Files ? 109
Lines ? 17901
Branches ? 0
===============================================
Hits ? 14828
Misses ? 3073
Partials ? 0 Continue to review full report at Codecov.
|
@gpucibot merge |
This is a non-functional documentation update.
The output of the c column is missing in the result of the first example in agg method:
python/cudf/cudf/core/groupby/groupby.py
I added the values of the c columns.