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.

Sign up for GitHub

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

Jump to bottom

[FEA] A cudf.config module to manage configuration options #5311

Closed
shwina opened this issue May 28, 2020 · 8 comments · Fixed by #11193
Closed

[FEA] A cudf.config module to manage configuration options #5311

shwina opened this issue May 28, 2020 · 8 comments · Fixed by #11193
Assignees
@shwina
Labels
feature request New feature or request Python Affects Python cuDF API.

Comments

@shwina
Copy link
Contributor

shwina commented May 28, 2020

In a discussion with @jakirkham the idea of a cudf.config module, similar to Dask was brought up.

The specific use case we had in mind was a way for users to specify whether or not to use pack/unpack serialization; but there may be other such "global" options that we may want to expose in the future.
@shwina shwina added feature request New feature or request Needs Triage Need team to review and classify Python Affects Python cuDF API. labels May 28, 2020
@shwina shwina removed the Needs Triage Need team to review and classify label May 28, 2020
@jakirkham
Copy link
Member

One other idea Ashwin mentioned was configuring the RMM allocator used.

@brandon-b-miller
Copy link
Contributor

+1 - could have an option here to disable/enable cross joins when implemented.

@shwina shwina self-assigned this Jun 1, 2020
@jakirkham
Copy link
Member

cc @rjzamora (in case you have needs for this as well)

@jakirkham jakirkham mentioned this issue Jun 16, 2020
Closed
@github-actions
Copy link

This issue has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.

@github-actions
Copy link

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

@jakirkham jakirkham mentioned this issue Mar 15, 2021
Closed
@jakirkham jakirkham mentioned this issue May 10, 2021
Closed
@jakirkham jakirkham mentioned this issue Jun 22, 2021
Merged
@charlesbluca
Copy link
Member

With #8153 merged, this configuration module could also be used to enable/disable packing of tables during serialization

@quasiben
Copy link
Member

quasiben commented Jul 1, 2021

Would it make sense if this option existed at the dask-cuda level ?

@quasiben
Copy link
Member

quasiben commented Jul 1, 2021

Ah, @charlesbluca pointed me to the previous dask-cuda discussion here:
rapidsai/dask-cuda#601

@charlesbluca charlesbluca mentioned this issue Jul 6, 2021
Closed
@shwina shwina mentioned this issue Mar 31, 2022
Open
@isVoid isVoid mentioned this issue Jul 1, 2022
Merged
rapids-bot bot pushed a commit that referenced this issue Jul 28, 2022
@isVoid
Add cudf.options (#11193)
e4ce301 
This PR adds `cudf.options`, a global dictionary to store configurations. A set of helper functions to manage the registries are also included. See documentation included in the PR for detail.

See demonstration use in: #11272

Closes #5311

Authors:
  - Michael Wang (https://github.com/isVoid)

Approvers:
  - Lawrence Mitchell (https://github.com/wence-)
  - Matthew Roeschke (https://github.com/mroeschke)
  - Bradley Dice (https://github.com/bdice)

URL: #11193
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shwina shwina

Labels
feature request New feature or request Python Affects Python cuDF API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add cudf.options isVoid/cudf
5 participants
@quasiben @jakirkham @shwina @charlesbluca @brandon-b-miller