-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[Shape Inference] Add shape inference for QLinearAdd and QLinearMul ops #24090
[Shape Inference] Add shape inference for QLinearAdd and QLinearMul ops #24090
Conversation
|
06588a2
to
0e88406
Compare
@microsoft-github-policy-service agree company="AMD" |
Signed-off-by: Praveen G <[email protected]>
0e88406
to
22a7f9f
Compare
You should merge main branch in your PR. Some changes are related to the failing pipeline which does not fail because of your changes. |
/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline,ONNX Runtime Web CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline |
/azp run Linux QNN CI Pipeline,onnxruntime-binary-size-checks-ci-pipeline,Big Models,Linux Android Emulator QNN CI Pipeline,Android CI Pipeline,iOS CI Pipeline,ONNX Runtime React Native CI Pipeline,Linux DNNL CI Pipeline,Linux MIGraphX CI Pipeline,Linux ROCm CI Pipeline |
Azure Pipelines successfully started running 8 pipeline(s). |
Azure Pipelines successfully started running 10 pipeline(s). |
@xadupre Thanks for the review. I suppose that I do not have required access to merge the PR. Can you help with this ? |
It is unexpected. It is your branch so you be able to do so. |
Yes, I am not sure if there are additional accesses required to merge into main branch or if it is due to the 3 failing checks. I do not see the merge option at all. |
Description
Support shape inference for QLinearAdd and QLinearMul ops which were missing in symbolic_shape_infer.py
Motivation and Context
This change is required to enable shape inference for models with "QLinearAdd" ops which are defined in com.microsoft domain and the shapes of which cannot be inferred using onnx shape_inference alone.
Fixes issue #24028