-
Notifications
You must be signed in to change notification settings - Fork 915
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
Remove unnecessary nunique function in Series. #10205
Remove unnecessary nunique function in Series. #10205
Conversation
Can one of the admins verify this patch? |
ok to test |
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.
Ack @martinfalisse and this is why we don't force push :) Thank you for the quick fix!
Codecov Report
@@ Coverage Diff @@
## branch-22.04 #10205 +/- ##
================================================
+ Coverage 10.42% 10.47% +0.05%
================================================
Files 119 122 +3
Lines 20603 20501 -102
================================================
Hits 2148 2148
+ Misses 18455 18353 -102
Continue to review full report at Codecov.
|
@gpucibot merge |
Removes unnecessary nunique function in Series, as indicated by @vyasr here #10077 (comment).
Seem to have made a mistake when correcting the style of my last commit and not have kept the original changes of that commit.