Skip to content
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

[FEA] Expose seed argument for series.hash_values #12775

Closed
ayushdg opened this issue Feb 14, 2023 · 0 comments · Fixed by #12795
Closed

[FEA] Expose seed argument for series.hash_values #12775

ayushdg opened this issue Feb 14, 2023 · 0 comments · Fixed by #12795
Labels
feature request New feature or request

Comments

@ayushdg
Copy link
Member

ayushdg commented Feb 14, 2023

Is your feature request related to a problem? Please describe.
libcudf and cudf cython allow for a seed param when computing the hash of a column. Which isn't exposed to the python api today.

Describe the solution you'd like
I have a use case which involves computing multiple hashes on a cudf column and having the ability to call hash_values with different seeds would help computing the different hashes I need.

Describe alternatives you've considered
N/A

Additional context
N/A

@ayushdg ayushdg added feature request New feature or request Needs Triage Need team to review and classify labels Feb 14, 2023
rapids-bot bot pushed a commit that referenced this issue Feb 24, 2023
This PR exposes the `seed` param to `hash_values` that is already supported by libcudf's `hash` method. 
Closes #12775

Authors:
  - Ayush Dattagupta (https://github.com/ayushdg)

Approvers:
  - https://github.com/brandon-b-miller

URL: #12795
@bdice bdice removed the Needs Triage Need team to review and classify label Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants