Skip to content

Conversation

@mdboom
Copy link
Contributor

@mdboom mdboom commented Oct 14, 2025

This is the script I used to update to a new CTK version and create #1131. I think this will be helpful going forward. Worth adding to toolshed?

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Oct 14, 2025

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@mdboom mdboom requested a review from Copilot October 14, 2025 14:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a script to automate the process of updating cuda-python to support a new version of the CUDA Toolkit (CTK). The script streamlines the multi-step process that was previously done manually.

  • Automates CTK version updates by fetching headers, updating configuration files, and running code generation
  • Handles both latest and previous version updates through command-line options
  • Provides clear instructions and confirmations for manual steps that still need to be performed

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@rwgk
Copy link
Collaborator

rwgk commented Oct 14, 2025

Looks useful! But I think it's important to have cybind in the name, maybe update_ctk_cybind.py?

This is to distinguish from cython-gen (which already has a regenerate.py script).

@mdboom
Copy link
Contributor Author

mdboom commented Oct 14, 2025

This is to distinguish from cython-gen (which already has a regenerate.py script).

Hmm... I actually didn't think of cython-gen here for this one because the last time this was manually done (in #960), none of the cython-gen-generated files needed updating. But for completeness, this should probably /also/ regenerate with cython-gen (by running that regenerate.py script). That's a trickier thing to automate because it requires /installing/ the CTK, rather than just fetching headers from the web, as this does.

What do others think?

@rwgk
Copy link
Collaborator

rwgk commented Oct 14, 2025

That's a trickier thing to automate because it requires /installing/ the CTK, rather than just fetching headers from the web, as this does.

For this time around, I'll try on the colossus workstation that I have already. I think it'll take me only a few minutes.

Not sure what'll be best long term.

@mdboom
Copy link
Contributor Author

mdboom commented Oct 14, 2025

Moving elsewhere.

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.

2 participants