[CAPI] remove legacy CAPI source code and document #21916
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
github-actions
bot
added
category: build
riverlijunjie
force-pushed
the
river/remove_legacy_capi_src
branch
from
a6e5fee to
fb8818e
Compare
riverlijunjie
requested review from
ilya-lavrenov,
zhaixuejun1993 and
vurusovs
zhaixuejun1993
approved these changes
Jan 3, 2024
vurusovs
reviewed
Jan 5, 2024
| @@ -145,11 +145,3 @@ int main_new() { | |||
| return 0; | |||
| } | |||
|
|
|||
| int main_old() { | |||
There was a problem hiding this comment.
Looks like all migration files may be removed
@ilya-lavrenov please, comment. If yes, let's remove it in next PR
There was a problem hiding this comment.
yes, they should be removed
vurusovs
reviewed
Jan 5, 2024
| @@ -29,7 +29,7 @@ OpenVINO C API has the following structure: | |||
| * [src](./src) contains the implementations of all C APIs. | |||
| * [tests](./tests) contains all tests for OpenVINO C APIs. [Learn more](./docs/how_to_write_unit_test.md). | |||
|
|
|||
| > **NOTE**: Using API 2.0 is strongly recommended. Legacy API (for C) [header file](./include/c_api/ie_c_api.h), [source file](./src/ie_c_api.cpp), [unit test](./tests/ie_c_api_test.cpp) are also included in the component, but the legacy API is no longer extended. | |||
| > **NOTE**: Using API 2.0 is strongly recommended. Legacy API (for C) has been removed from 2024.0. | |||
There was a problem hiding this comment.
Suggested change
| > **NOTE**: Using API 2.0 is strongly recommended. Legacy API (for C) has been removed from 2024.0. | |
| > **NOTE**: Legacy Inference Engine API (for C) has been removed from 2024.0. |
vurusovs
approved these changes
Jan 5, 2024
vurusovs
reviewed
Jan 5, 2024
|
|
||
| ## OV Core | ||
|
|
||
| This strcut represents OpenVINO entity and allows you to manipulate with plugins using unified interfaces. |
There was a problem hiding this comment.
Suggested change
| This strcut represents OpenVINO entity and allows you to manipulate with plugins using unified interfaces. | |
| This struct represents OpenVINO entity and allows you to manipulate with plugins using unified interfaces. |
vurusovs
reviewed
Jan 5, 2024
|
|
||
| - Description: Loads a network that was read from the Intermediate Representation (IR) to the plugin with specified device name and creates an `ie_executable_network_t` instance of the `ie_network_t` struct. | ||
| You can create as many networks as you need and use them simultaneously (up to the limitation of the hardware resources). | ||
| - Description: Reads models from IR / ONNX / PDPD / TF / TFLite formats to create ov_model_t. |
There was a problem hiding this comment.
Suggested change
| - Description: Reads models from IR / ONNX / PDPD / TF / TFLite formats to create ov_model_t. | |
| - Description: Reads models from IR / PyTorch / ONNX / PDPD / TF / TFLite formats to create ov_model_t. |
There was a problem hiding this comment.
Will update these documentation issued in the next PR.
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:
Tickets: