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.

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

Replace Docker images for CUDA 12.5 with CUDA 12.8 #573

Merged
merged 5 commits into from
Feb 12, 2025
Merged

Conversation

caryr35
Copy link
Contributor

@caryr35 caryr35 commented Feb 3, 2025

RAPIDS is planning to deprecate support of CUDA 12.5 Docker containers in Release v25.02, and to cease publishing CUDA 12.5 Docker containers before the v25.02 release. The CUDA 12.5 containers will be replaced by CUDA 12.8 containers.

@caryr35 caryr35 requested a review from a team as a code owner February 3, 2025 21:02
Copy link

vercel bot commented Feb 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Feb 12, 2025 9:53pm

Copy link

netlify bot commented Feb 3, 2025

Deploy Preview for docs-rapids-ai ready!

Name Link
🔨 Latest commit 6956469
🔍 Latest deploy log https://app.netlify.com/sites/docs-rapids-ai/deploys/67ad1813393a690008369cf0
😎 Deploy Preview https://deploy-preview-573--docs-rapids-ai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bdice bdice changed the title Deprecated CUDA 12.5 & Upgrade to CUDA 12.8 Deprecate Docker images for CUDA 12.5 & Upgrade to CUDA 12.8 Feb 3, 2025
Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small change suggestion, otherwise looks good.

bdice
bdice previously requested changes Feb 3, 2025
Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@caryr35 You'll need to rename this to RSN 42 -- we already have an RSN 41 from #566.

@jakirkham
Copy link
Member

Maybe we should use the word "Sunset" or "Remove" instead of "Deprecate". It sounds like they will be completely gone as opposed to going through a deprecation cycle

Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Cary! 🙏

Have added a couple suggestions based on my understand of our goals here

Please let me know what you think 🙂

@jakirkham jakirkham changed the title Deprecate Docker images for CUDA 12.5 & Upgrade to CUDA 12.8 Replace Docker images for CUDA 12.5 with CUDA 12.8 Feb 12, 2025
Copy link
Member

@jakirkham jakirkham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks Cary! 🙏

@jakirkham jakirkham requested a review from bdice February 12, 2025 22:55
@jakirkham jakirkham dismissed bdice’s stale review February 12, 2025 23:07

RSN number has been updated

@jakirkham
Copy link
Member

Looks like Bradley is out for the rest of the week. Going to go ahead and merge. We can follow up on anything else after

@jakirkham jakirkham removed the request for review from bdice February 12, 2025 23:10
@jakirkham jakirkham merged commit 349409a into main Feb 12, 2025
6 checks passed
@jakirkham jakirkham deleted the cuda-12.8-upgrade branch February 12, 2025 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants