-
Notifications
You must be signed in to change notification settings - Fork 156
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
Use branch-23.12 workflows. #1284
Use branch-23.12 workflows. #1284
Conversation
/merge |
@bdice looks like there is an |
@harrism I fixed the remaining 23.10 references on the 23.12 branch (not related to this PR as far as I could see). We'll need a Python codeowner review to merge. |
@@ -113,7 +113,7 @@ An example command from the Release Selector: | |||
docker run --gpus all --pull always --rm -it \ | |||
--shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864 \ | |||
-p 8888:8888 -p 8787:8787 -p 8786:8786 \ | |||
nvcr.io/nvidia/rapidsai/notebooks:23.120-cuda11.8-py3.10 | |||
nvcr.io/nvidia/rapidsai/notebooks:23.12-cuda11.8-py3.10 |
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.
The update-version.sh
regex for the README had an error, which I fixed.
@@ -26,7 +26,7 @@ requires = [ | |||
|
|||
[project] | |||
name = "cuproj" | |||
version = "23.12.0" | |||
version = "23.12.00" |
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.
Aligning with other RAPIDS projects -- these lines are supposed to be "full versions" and not "PEP 440 versions." I'm not sure how this typo occurred, because the update-version.sh
script is set to do this correctly as far as I can tell.
This PR switches back to using
branch-23.12
for CI workflows because the CUDA 12 ARM conda migration is complete.