-
Notifications
You must be signed in to change notification settings - Fork 890
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
Fix typo in Series.value_counts
#14550
Fix typo in Series.value_counts
#14550
Conversation
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 note, I would love to be able to return things of type cudf::size_type
here.
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.
Would also be good to have a unit test hitting this path
We do have |
Tests were failing already due to this issue, they will now start to pass. |
Yeah, I meant that I would love not to have to match pandas and return an int64 when we know int32 (or rather size_type) must suffice |
a068b10
into
rapidsai:pandas_2.0_feature_branch
Description
This PR fixes the return type of
Series.value_counts
to returnint64
, correcting a typo that wasint34
.Checklist