-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Support quantised SQRT operator in TFLite #9258
Conversation
cc @ophirfrish @ekalda @mbaret who may also be interested |
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.
The change looks good to me.
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.
@elinx can you re-run CI on this PR so that we can try to merge it once reviewers are happy?
d53b8db
to
58434f2
Compare
58434f2
to
5213fa5
Compare
@leandron I have rebased the code and passed the CI, please take a look. |
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, thanks @elinx
This is merged now, thanks @elinx, @lhutton1 and @ophirfrish! |
Supoort SQRT quantization operator convertion as a part of tasks from issue #9187
cc @leandron