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

[QNN EP] Re-enable several disabled QNN-EP UTs #23799

Merged
merged 4 commits into from
Feb 26, 2025

Conversation

kuanyul-quic
Copy link
Contributor

@kuanyul-quic kuanyul-quic commented Feb 24, 2025

Description

  1. Re-enable UTs which passed 2.30
  2. Update resize UT because "round_prefer_floor" is no longer supported in QNN SDK since 2.21.

Motivation and Context

  1. Make the UT of QNN EP pass as much as possible to improve the test coverage.

### Description
1. Re-enable UTs which passed 2.30
2. Fix conv and resize UTs
   - Make conv's weight as initializer to let graph.NumberOfNodes()
     match ep_nodes, which should be 1.
   - Update resize UT because "round_prefer_floor" is no longer
     supported in QNN SDK since 2.21.

### Motivation and Context
1. Make the UT of QNN EP pass as much as possible to improve the test
   coverage.
@kuanyul-quic
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Qualcomm"

@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Linux Android Emulator QNN CI Pipeline

@HectorSVC HectorSVC added the ep:QNN issues related to QNN exeution provider label Feb 24, 2025
kuanyul-quic and others added 3 commits February 25, 2025 17:20
### Description
1. Re-enable UTs which passed 2.30
2. Update resize UT because "round_prefer_floor" is no longer supported
   in QNN SDK since 2.21.

### Motivation and Context
1. Make the UT of QNN EP pass as much as possible to improve the test
   coverage.
…nnxruntime into dev/kuanyul/enable_qnnep_ut
@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Linux Android Emulator QNN CI Pipeline

Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@HectorSVC
Copy link
Contributor

/azp run Big Models,Win_TRT_Minimal_CUDA_Test_CI,Windows CPU CI Pipeline,Windows GPU CUDA CI Pipeline,Windows GPU DML CI Pipeline,Windows GPU Doc Gen CI Pipeline,Windows GPU TensorRT CI Pipeline

@HectorSVC
Copy link
Contributor

/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, onnxruntime-binary-size-checks-ci-pipeline

Copy link

Azure Pipelines successfully started running 7 pipeline(s).

1 similar comment
Copy link

Azure Pipelines successfully started running 7 pipeline(s).

@HectorSVC
Copy link
Contributor

/azp run Linux OpenVINO CI Pipeline

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@HectorSVC HectorSVC merged commit 5be82eb into microsoft:main Feb 26, 2025
77 checks passed
guschmue pushed a commit that referenced this pull request Mar 6, 2025
### Description
1. Re-enable UTs which passed 2.30
2. Update resize UT because "round_prefer_floor" is no longer supported in QNN SDK since 2.21.

### Motivation and Context
1. Make the UT of QNN EP pass as much as possible to improve the test
coverage.

---------

Co-authored-by: Kuan-Yu Lin <kuanyul@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ep:QNN issues related to QNN exeution provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants