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

Enable generic feature level devices in DML EP #20114

Merged
merged 4 commits into from
Mar 29, 2024

Conversation

jeffbloo
Copy link
Contributor

@jeffbloo jeffbloo commented Mar 28, 2024

Description

Enable NPUs supporting DXCORE_ADAPTER_ATTRIBUTE_D3D12_GENERIC_ML and D3D_FEATURE_LEVEL_1_0_GENERIC with DML EP. This also begins ingesting DX headers through the DirectX-Headers repo.

Note that this includes an update to cgamanifest.json for onnx-tensorrt which is triggered during re-generation due to a prior changes to deps.txt.

Motivation and Context

@jeffbloo jeffbloo requested a review from smk2007 March 28, 2024 00:20
@jeffbloo jeffbloo changed the title User/jeffbloo/dml generic device 1 Enable generic feature level devices in DML EP Mar 28, 2024
@jeffbloo jeffbloo requested review from a team as code owners March 29, 2024 01:00
@jeffbloo jeffbloo requested a review from chilo-ms March 29, 2024 01:01
@jeffbloo jeffbloo merged commit 2f31560 into main Mar 29, 2024
95 checks passed
@jeffbloo jeffbloo deleted the user/jeffbloo/DmlGenericDevice-1 branch March 29, 2024 21:37
TedThemistokleous pushed a commit to TedThemistokleous/onnxruntime that referenced this pull request May 7, 2024
### Description
Enable NPUs supporting DXCORE_ADAPTER_ATTRIBUTE_D3D12_GENERIC_ML and
D3D_FEATURE_LEVEL_1_0_GENERIC with DML EP. This also begins ingesting DX
headers through the DirectX-Headers repo.

Note that this includes an update to cgamanifest.json for onnx-tensorrt
which is triggered during re-generation due to a prior changes to
deps.txt.

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
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.

4 participants