-
Notifications
You must be signed in to change notification settings - Fork 240
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
[OV] Introduce support of quantization If operation #2101
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #2101 +/- ##
===========================================
- Coverage 36.24% 36.09% -0.15%
===========================================
Files 477 478 +1
Lines 42570 42757 +187
===========================================
+ Hits 15428 15432 +4
- Misses 27142 27325 +183
|
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.
Need to have explanations for each method that was added. Also, the changes for the PostTrainingQuantization
are not clear, including adding the backend entity for it.
The general approach and reasons for it are not clear.
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
Changes
Add support of quantization for OpenVINO models with If operation - https://docs.openvino.ai/2023.0/openvino_docs_ops_infrastructure_If_8.html
Reason for changes
Customer request / Obtain the maximum performance for models with inner subgraphs under If operation
Related tickets
113826
Tests
Tested on a model attached to a ticket
Add synthetic model with If operation