-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implement dequantizeLinear #134
Conversation
c75ee80
to
2bfbb81
Compare
2bfbb81
to
2b97724
Compare
7a4f312
to
a97dfb2
Compare
You'll need to rebase the code due to my latest force push, thanks! |
2b97724
to
5f0efb1
Compare
@huningxin Please take another look. Thanks! |
@huningxin Please hold on. I will upload a new commit to fix a bug. |
@huningxin ready for review again. Thanks! |
c6bc4f9
to
92c4e15
Compare
@huningxin Uploaded a new commit to address the remaining comments. Please take another look. Thanks! |
@huningxin Please take another look. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a nit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The implementation of dequantizelinear is a little confusion since currently WebNN spec definition for dequantizelinear is wildly different from ONNX. Besides, OpenVINO also have many constraints.