Skip to content

Conversation

@nnethercote
Copy link
Collaborator

In addition to the 12.8.1 testing.

@nnethercote nnethercote marked this pull request as draft December 1, 2025 04:47
@nnethercote nnethercote force-pushed the cuda13-on-windows-ci branch 27 times, most recently from cc3ac83 to a3f5617 Compare December 2, 2025 05:21
@nnethercote nnethercote marked this pull request as ready for review December 2, 2025 05:45
@nnethercote nnethercote requested a review from LegNeato December 2, 2025 05:45
@nnethercote
Copy link
Collaborator Author

After a bazillion rounds of trial and error this is now working.

cuda: ${{ matrix.cuda }}
method: network
linux-local-args: ${{ toJson(matrix.linux-local-args) }}
linux-local-args: "[]"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be deleted like the other one? It doesn't appear to be used?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

True, https://github.com/Jimver/cuda-toolkit says it's Linux only. I will remove it.

- `pwsh` is the default shell on Windows, no need to specify it.
- Putting a backtick at the end of a line lets you continue it on the
  next, like a backslash in Unix shells.
We need to account for some CUDA 12 vs 13 differences:
- new sub-packages
- the location of `nvvm64_40_0.dll`
- Jitver/cuda-toolkit needs updating to handle CUDA 13
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