Changes to the OpenVINO 2.0 Transition Guide #10936
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Let's merge #10933 first |
MaximProshin
approved these changes
Mar 14, 2022
|
@andrew-zaytsev please, see the latest version and check whether some changes are still needed. |
…azaytsev/transition-guide-fixes # Conflicts: # docs/OV_Runtime_UG/migration_ov_2_0/intro.md
andrew-zaytsev
added a commit
to andrew-zaytsev/openvino
that referenced
this pull request
Mar 15, 2022
andrew-zaytsev
added a commit
that referenced
this pull request
Mar 15, 2022
| - Model Optimizer changed layouts for TensorFlow models (see [Layouts in OpenVINO](../layout_overview.md)). It leads to unexpected user behavior that a user needs to use a different layout for its input data with compare to the framework: | ||
| ![tf_openvino] | ||
| - Inference Engine API (`InferenceEngine::CNNNetwork`) also applied some conversion rules for input and output precisions because of device plugins limitations. | ||
| - Users need to specify input shapes during model conversions in Model Optimizer and work with static shapes in the application. | ||
|
|
||
| OpenVINO API 2.0 is introduced to align logic of working with model as it is done in the frameworks - no layout and precision changes, operates with tensor names and indeces to address inputs and outputs. OpenVINO Runtime is composed of Inference Engine API used for inference and ngraph API targeted to work with models, operations. The API 2.0 has common structure, naming convention styles, namespaces, removes duplicated structures. See [How to migrate to API 2.0](./common_inference_pipeline.md) for details. | ||
| OpenVINO™ introduces API 2.0 to align logic of working with model as it is done in the frameworks - no layout and precision changes, operates with tensor names and indices to address inputs and outputs. OpenVINO Runtime is composed of Inference Engine API used for inference and nGraph API targeted to work with models, operations. The API 2.0 has common structure, naming convention styles, namespaces, removes duplicated structures. See [How to migrate to OpenVINO API v2](common_inference_pipeline.md) for details. |
There was a problem hiding this comment.
why you changed to v2?? In other places of the document we have 2.0
CC @xu-yuan1 what is the proper name?
There was a problem hiding this comment.
The approved names by marketing are "API 2.0", "OpenVINO API v2" and "OpenVINO 2022.1". I think "OpenVINO API 2.0" is in a grey area as it's not forbidden explicitly. @andrew-zaytsev Andrey may know better and confirm.
But I think on the same page we should keep consistent, having both "2.0" and "v2" can be confusing. Or maybe we can have a statement at the beginning to say that they are the same thing.
|
@andrew-zaytsev please, port these changes to master if needed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Details:
Minor changes to the OpenVINO 2.0 Transition Guide
Tickets:
CVS-80835