Skip to content

Conversation

@yuhuchua-qti
Copy link
Contributor

Description

This PR fixes the following UT failures:

  • Gemm_Dynamic_A_B_Bias
  • Gemm_Static_B_And_Bias
  • Gemm_TransAB_Static_B_And_Bias
  • Gemm_TransAB_Dynamic_B_And_Bias
  • Gemm_Broadcast_Bias_DynamicInputs
  • Gemm_Broadcast_Bias_DynamicA_StaticB_DynamicC
  • Resize_DownSample_Linear_HalfPixel_scales
  • EinsumRank3MatMul_QK

It also adds explanatory comments for MaxPool tests that must remain disabled due to QNN CPU's lack of ceil_mode support:

  • DISABLED_MaxPool_Ceil
  • DISABLED_MaxPool_Large_Input2_Ceil

Motivation and Context

Several unit tests for Gemm, Resize, and Einsum were failing under the QNN EP. These failures need to be resolved to ensure operator correctness and maintain test coverage. Additionally, some MaxPool tests using ceil_mode cannot be supported by QNN CPU, and the absence of clarifying comments may cause confusion for future maintainers.

…or unsupported MaxPool tests

Motivation:
Several unit tests for Gemm, Resize, and Einsum were failing under the
QNN EP. These failures need to be resolved to ensure operator
correctness and maintain test coverage. Additionally, some MaxPool tests
using ceil_mode cannot be supported by QNN CPU, and the absence of
clarifying comments may cause confusion for future maintainers.

Description:
This PR fixes the following UT failures:
  - Gemm_Dynamic_A_B_Bias
  - Gemm_Static_B_And_Bias
  - Gemm_TransAB_Static_B_And_Bias
  - Gemm_TransAB_Dynamic_B_And_Bias
  - Gemm_Broadcast_Bias_DynamicInputs
  - Gemm_Broadcast_Bias_DynamicA_StaticB_DynamicC
  - Resize_DownSample_Linear_HalfPixel_scales
  - EinsumRank3MatMul_QK

It also adds explanatory comments for MaxPool tests that must remain disabled
due to QNN CPU's lack of ceil_mode support:
  - DISABLED_MaxPool_Ceil
  - DISABLED_MaxPool_Large_Input2_Ceil
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.

1 participant