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

DML incorrect results, probably Split node #21007

Open
jlami opened this issue Jun 11, 2024 · 2 comments
Open

DML incorrect results, probably Split node #21007

jlami opened this issue Jun 11, 2024 · 2 comments
Labels
ep:DML issues related to the DirectML execution provider ep:OpenVINO issues related to OpenVINO execution provider platform:windows issues related to the Windows platform

Comments

@jlami
Copy link

jlami commented Jun 11, 2024

Describe the issue

Running the htdemucs model on DML gives incorrect results compared to the CPU execution.

To reproduce

The model we are using is the htdemucs v4 model from here: https://github.com/facebookresearch/demucs

You can find a converted onnx model in this zip: https://github.com/intel/openvino-plugins-ai-audacity/releases/download/v3.4.2-R1/openvino-models.zip (2GB zip, htdemucs_v4.onnx itself is 170MB)

If I comment out this line, DML gives correct results:

{REG_INFO_VER( 11, Split, typeNameListDefault, supportedTypeListAllScalars, DmlGraphSupport::Supported)}, // Adds negative axis.

Urgency

No response

Platform

Windows

OS Version

10

ONNX Runtime Installation

Built from Source

ONNX Runtime Version or Commit ID

74028e4

ONNX Runtime API

C

Architecture

X64

Execution Provider

DirectML

Execution Provider Library Version

1.14

@github-actions github-actions bot added ep:DML issues related to the DirectML execution provider ep:OpenVINO issues related to OpenVINO execution provider platform:windows issues related to the Windows platform labels Jun 11, 2024
@jlami
Copy link
Author

jlami commented Jun 12, 2024

Setting ep.dml.disable_graph_fusion to 1 while enabling the Split op again also gives correct results, so I guess the problem is in the DmlGraphFusionTransformer

@sophies927
Copy link
Contributor

@smk2007

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ep:DML issues related to the DirectML execution provider ep:OpenVINO issues related to OpenVINO execution provider platform:windows issues related to the Windows platform
Projects
None yet
Development

No branches or pull requests

2 participants