Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding DML to python cuda package #22606

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

Adding DML to python cuda package #22606

wants to merge 62 commits into from

Conversation

jchen351
Copy link
Contributor

@jchen351 jchen351 commented Oct 25, 2024

Description

Adding a new parameter --devices for the create_backend_test function in onnxruntime/test/python/onnx_backend_test_series.py to allow specifying the devices to test.

Because both tests and supported EPs are in exclusion lists. We only run tests that is compatible for all devices from args.devices, and EPs that are defined.

In case when only CUDA or DML EP are enabled, Python packaging tests will only run the tests that is compatible with the enabled EP, and only the enabled EP and CPU EP will be enabled. CPU EP is always enabled.

In case when both CUDA and DML EP are enabled. Python packaging tests will only run the tests that is compatible with both CUDA and DML, and only CPU EP will be enabled (CUDA and DML EPs are mutually exclusive). CPU EP is always enabled.

Motivation and Context

@jchen351 jchen351 requested a review from a team as a code owner October 25, 2024 17:15
@jchen351 jchen351 changed the title Adding DMK to python cuda package Adding DML to python cuda package Oct 25, 2024
@jchen351 jchen351 closed this Nov 11, 2024
@jchen351 jchen351 reopened this Nov 11, 2024
@snnn
Copy link
Member

snnn commented Nov 19, 2024

Please run the pipelines you modified.

@snnn
Copy link
Member

snnn commented Nov 19, 2024

Previously we saw after this change "Python-CUDA-Packaging-Pipeline" always timed out. Was the issue resolved or not?

@jchen351 jchen351 requested a review from snnn November 22, 2024 18:37
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