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

[ONNX] Added DequantizedLinear from com.microsoft domain #28716

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

vatsalashanubhag
Copy link
Contributor

Details:
Microsoft Contrib Operator "DequantizedLinear" for ONNX RT

Tickets:
N/A

@github-actions github-actions bot added the category: ONNX FE OpenVINO ONNX FrontEnd label Jan 28, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Jan 28, 2025
@gkrivor
Copy link
Contributor

gkrivor commented Jan 28, 2025

@vatsalashanubhag hi! Thanks for contribution! I see a fastgelu implementation here, lokks like by mistake.

and i don’t think I get why we need dequantizelinear as a separate implementation instead of continue usage of

ONNX_OP_M("DequantizeLinear", OPSET_SINCE(1), ai_onnx::opset_13::dequantize_linear, MICROSOFT_DOMAIN);

Could you check/highlight difference?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: ONNX FE OpenVINO ONNX FrontEnd ExternalPR External contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants