Skip to content

Make Windows CUDA Python tests deterministic - #2478

Open
Tianlei Wu (tianleiwu) with Copilot wants to merge 1 commit into
mainfrom
copilot/improve-windows-cuda-ci
Open

Make Windows CUDA Python tests deterministic#2478
Tianlei Wu (tianleiwu) with Copilot wants to merge 1 commit into
mainfrom
copilot/improve-windows-cuda-ci

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

The Windows CUDA job could auto-select an unintended execution provider or fall back to CPU. Its sccache path was also scoped incorrectly, preventing cache persistence.

  • CUDA validation
    • Check CUDA support before downloading test models.
    • Explicitly run the Python test driver with --eps cuda.
python test/python/test_onnxruntime_genai.py `
  --eps cuda `
  --e2e
  • sccache persistence
    • Define SCCACHE_DIR at workflow scope.
    • Remove the redundant build-step assignment so restore, build, and save use the same directory.

Co-authored-by: tianleiwu <30328909+tianleiwu@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@tianleiwu
Tianlei Wu (tianleiwu) marked this pull request as ready for review August 27, 2026 17:26
Copilot AI lite review requested due to automatic review settings August 27, 2026 17:26
@tianleiwu
Tianlei Wu (tianleiwu) enabled auto-merge (squash) August 27, 2026 17:26

Copilot AI 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.

Pull request overview

This PR updates the Windows CUDA CI workflow to make Python testing deterministic by forcing CUDA execution-provider selection and to improve sccache persistence by scoping SCCACHE_DIR at the workflow level.

Changes:

  • Define SCCACHE_DIR at workflow scope and remove the redundant per-step assignment.
  • Add a CUDA EP verification step before running Python tests.
  • Run the Python test driver with --eps cuda to avoid auto-selecting an unintended EP / CPU fallback.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/win-cuda-x64-build.yml
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