Skip to content

Source install selector from platform support data#802

Open
bdice wants to merge 6 commits into
rapidsai:mainfrom
bdice:codex/selector-platform-support-data
Open

Source install selector from platform support data#802
bdice wants to merge 6 commits into
rapidsai:mainfrom
bdice:codex/selector-platform-support-data

Conversation

@bdice

@bdice bdice commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • render platform support YAML directly into the install selector through Jekyll
  • derive Python versions, CUDA majors, default selections, and Conda toolkit bounds from that data
  • consolidate the separate stable/nightly and Conda CUDA declarations
  • add a pre-commit validator for release coverage, ordering, uniqueness, and toolkit bounds

Why

The install selector duplicated platform support values across several hardcoded arrays and a separate Conda CUDA bounds object. Serializing the existing Jekyll data keeps _data/platform_support.yml as the single source for supported Python and CUDA values without introducing another generated JSON artifact.

Validation

  • pre-commit run --all-files
  • rendered selector JavaScript syntax check with Node.js
  • stable/nightly runtime assertions for release labels and Conda, pip, and Docker commands

@netlify

netlify Bot commented Jun 30, 2026

Copy link
Copy Markdown

Deploy Preview for docs-rapids-ai ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9f669fc
🔍 Latest deploy log https://app.netlify.com/projects/docs-rapids-ai/deploys/6a444ec2c0f102000835fdbd
😎 Deploy Preview https://deploy-preview-802--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 project configuration.

@bdice bdice changed the title [codex] Source install selector from platform support data Source install selector from platform support data Jun 30, 2026
@bdice bdice marked this pull request as ready for review June 30, 2026 19:12
@bdice bdice requested a review from a team as a code owner June 30, 2026 19:12

@rockhowse rockhowse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok did a quick review, just so I understand a bit more

  • #801

    • has been ✅ + :git-merged:
    • was a pre-req to provide data we are basically already using to test nightlies
      adds "nightly" data to the Platform Support
  • #802 was a fast follow

    • uses the _data/platform_support.yml‎ as the source of truth for both release and nightly (new)
    • removes the data embedded the selector
    • now includes a python helper script and pre-check to validate the metadata matches reality?
    • simplifies the logic internally on this page
    • Reviewed the render in the PR and it looks to be good and install selector is working as expected

Makes sense to me and is basically what I was trying to articulate we do for things like this during the build meeting.

@bdice bdice requested a review from jakirkham June 30, 2026 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants