-
Notifications
You must be signed in to change notification settings - Fork 901
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
Drop Pascal GPU support. #14630
Drop Pascal GPU support. #14630
Conversation
@vuule There are a couple places where the C++ code has special handling for disabling ZSTD compression on Pascal. Can you weigh in on whether that is safe to remove, since 24.02 will not support Pascal GPUs? cudf/cpp/tests/io/comp/decomp_test.cpp Line 216 in cee6429
cudf/cpp/src/io/comp/nvcomp_adapter.cpp Line 554 in cee6429
|
Should be safe to remove. I'd keep |
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.
README.md of
python/cudf/
python/cudf_kafka/
python/custreamz/
python/dask_cudf/
needs update as well.
@karthikeyann Can you check this? I already updated |
@karthikeyann I'm going to go ahead and merge this since we've had enough reviews and I think your comment is already addressed. If you see additional changes that are needed, please let me know and I'll follow up. 👍 |
/merge |
Description
Closes #14629.
Checklist