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

Clean up CUDA gpu_utils.py #10076

Closed
bdice opened this issue Jan 19, 2022 · 0 comments · Fixed by #10113
Closed

Clean up CUDA gpu_utils.py #10076

bdice opened this issue Jan 19, 2022 · 0 comments · Fixed by #10113
Assignees
@bdice
Labels
Python Affects Python cuDF API.

Comments

@bdice
Copy link
Contributor

bdice commented Jan 19, 2022

We can make some improvements to python/cudf/cudf/utils/gpu_utils.py now that we've dropped CUDA 10 compatibility. I noted some of these in comments on #10008.

Originally posted by @bdice in #10008 (comment)
@bdice bdice self-assigned this Jan 19, 2022
@bdice bdice added tech debt Python Affects Python cuDF API. labels Jan 19, 2022
This was referenced Jan 19, 2022
Merged
Merged
rapids-bot bot pushed a commit that referenced this issue Jan 26, 2022
@bdice
Update gpu_utils.py to reflect current CUDA support. (#10113)
05d53cf 
This PR resolves #10076. It improves `gpu_utils.py` by removing code for handling CUDA < 11.0, which we no longer support.

This is marked as "breaking" because of minor Python API changes. I changed the name of an error class from `UnSupportedCUDAError` to `UnsupportedCUDAError` and removed an unused error class named `UnSupportedGPUError`. It appears the `UnSupportedGPUError` class was introduced in #4692 but has never been used in the code.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Ashwin Srinath (https://github.com/shwina)
  - GALI PREM SAGAR (https://github.com/galipremsagar)

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

@bdice bdice

Labels
Python Affects Python cuDF API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update gpu_utils.py to reflect current CUDA support. bdice/cudf
1 participant
@bdice