-
Notifications
You must be signed in to change notification settings - Fork 300
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 support for PASCAL #4022
Comments
A list of files where we left FIXME for these issues:
|
rapids-bot bot
pushed a commit
that referenced
this issue
Dec 13, 2023
The only remaining "support" for Pascal was a check that if we're on a Pascal GPU we expect the function to throw an exception. Not sure it does anymore (don't see evidence of it, we haven't tested on Pascal for a long time), but if we're not supporting Pascal it doesn't matter. Closes #4022 Authors: - Chuck Hastings (https://github.com/ChuckHastings) Approvers: - Seunghwa Kang (https://github.com/seunghwak) - AJ Schmidt (https://github.com/ajschmidt8) - Rick Ratzel (https://github.com/rlratzel) URL: #4044
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
RAPIDS is dropping support for PASCAL in release 24.02. There are a collection of places in the C++ code where we check at runtime to see if we're executing on PASCAL and change the behavior (e.g. raise an exception, run a different code path).
We should review the code (I think it's all Louvain related) and drop all of the extra code for supporting PASCAL.
The text was updated successfully, but these errors were encountered: