-
Notifications
You must be signed in to change notification settings - Fork 31
Update rapids version to v25.10 #616
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
Conversation
bdice
left a comment
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.
source/conf.py also appears to need updates. Is there an update-version.sh script for this repo?
|
I think @AyodeAwe has a script that generates these PRs, not sure where it lives though. |
|
Reviews addressed @bdice, @jacobtomlinson |
There is but I don't think an update there is needed based on this initial round of review. |
bdice
left a comment
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.
More changes are needed like the suggestions below. Should some of the commands listed be using CUDA 13?
|
Generally in the deployment docs we just show the "default". So we might want to hold off on CUDA 13 for a while until there is a larger adoption of 13. |
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
|
@bdice I've committed your suggestions. We should be good here, right? |
Co-authored-by: Bradley Dice <bdice@bradleydice.com>
|
@AyodeAwe Yes, approved. |
Update rapids version to v25.10