DOCS Change optimization images#17421
DOCS Change optimization images#17421tsavina wants to merge 3 commits intoopenvinotoolkit:masterfrom
Conversation
|
@AlexKoff88 , please check updates in the diagrams |
There was a problem hiding this comment.
I cannot find where this diagram is used. Is this a POT-specific picture? The reason why I am asking is that if it is a common picture that POT should be replaced with NNCF.
There was a problem hiding this comment.
yes. sorry I might have accidentally closed the comment with explanation
it fixes POT description page https://docs.openvino.ai/latest/pot_introduction.html
|
Within this document (tmo_introduction.html) there is diagram that has model optimizer. We probably want to change it to convert modes or something similar. Not sure if this is covered by separate PR. cc: @slyalin |
There was a problem hiding this comment.
IR is just a serialized representation for OpenVINO model. As we are becoming more Pythonic, and moving away from CLI tool, here is a better option for names:
Convert to OpenVINO IR->Convert to OpenVINO ModelFull precision IR->Full precision OpenVINO ModelOptimized IR->Optimized OpenVINO Model
And, probably in this particular diagram, to emphasize separation between Development and Deployment the latest step in Development should be serialization of Optimized OpenVINO Model to Optimized OpenVINO IR. Some additional tuning for names is still required.
There was a problem hiding this comment.
This image says you need to convert your model to IR before you "optimize with NNCF" (in the PTQ part). This is not true for PTQ with NNCF (it was for POT).
There was a problem hiding this comment.
If you change Run Model Optimizer to Convert to OpenVINO IR in one of the pictures, then apply similar changes to this one.
Don't know about another PR. Adding @kblaszczak-intel and @andrei-kochin. The changes haven't been prepared yet. In other comments in this PR, I recommended updating pictures immediately as it has been already partially done. |
|
This PR will be closed in a week because of 2 weeks of no activity. |
Details:
Tickets: