Skip to content

Conversation

jrcichra
Copy link
Collaborator

Description

Switch the GPU to MCDM mode and run the test suite.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

Switch the GPU to MCDM mode and run the test suite.
@jrcichra jrcichra requested a review from leofang September 18, 2025 19:04
Copy link
Contributor

copy-pr-bot bot commented Sep 18, 2025

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

Contributors can view more details about this message here.

@jrcichra
Copy link
Collaborator Author

/ok to test

Copy link

@cpcloud
Copy link
Contributor

cpcloud commented Sep 18, 2025

Is there some motivation or context for this PR? Would be nice for the broader team to know what the motivation here is.

@leofang
Copy link
Member

leofang commented Sep 18, 2025

We are testing the windows runners, it's the easiest to use cuda-python as the guinea pig, see discussions in nv-gha-runners-windows.

@leofang
Copy link
Member

leofang commented Sep 18, 2025

Looks like I did not create a better issue than this: #462. We need to test all 3 modes on Windows ultimately: TCC, WDDM, MCDM.

@leofang leofang marked this pull request as ready for review September 19, 2025 00:54
Copy link
Contributor

copy-pr-bot bot commented Sep 19, 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.

@leofang leofang marked this pull request as draft September 19, 2025 00:54
@leofang
Copy link
Member

leofang commented Sep 19, 2025

/ok to test

@leofang
Copy link
Member

leofang commented Sep 19, 2025

It works!

@leofang
Copy link
Member

leofang commented Oct 5, 2025

Let's close this PR and revisit MCDM testing when we start working on #462/#985.

@leofang leofang closed this Oct 5, 2025
@leofang leofang deleted the jrcichra/mcdm branch October 5, 2025 21:28
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