Skip to content

Use runtime::Tensor instead of Blob in read_model#7951

Merged
ilya-lavrenov merged 3 commits intoopenvinotoolkit:masterfrom
ilyachur:fix_read_model_api
Oct 13, 2021
Merged

Use runtime::Tensor instead of Blob in read_model#7951
ilya-lavrenov merged 3 commits intoopenvinotoolkit:masterfrom
ilyachur:fix_read_model_api

Conversation

@ilyachur
Copy link
Copy Markdown
Contributor

@ilyachur ilyachur commented Oct 12, 2021

Details:

  • Changed read_model API

Tickets:

@ilyachur ilyachur added the category: inference OpenVINO Runtime library - Inference label Oct 12, 2021
@ilyachur ilyachur added this to the 2022.1 milestone Oct 12, 2021
@openvino-pushbot openvino-pushbot added the category: IE Tests OpenVINO Test: plugins and common label Oct 12, 2021
@ilyachur ilyachur requested a review from a team October 12, 2021 09:48
@ilya-lavrenov ilya-lavrenov self-assigned this Oct 12, 2021
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) October 12, 2021 10:02
Copy link
Copy Markdown
Contributor

@apankratovantonp apankratovantonp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ov::runtime::Tensor() -> {}

@ilya-lavrenov
Copy link
Copy Markdown
Contributor

ov::runtime::Tensor() -> {}

no, in this case we can call read_model(string, string) which is wrong

@apankratovantonp
Copy link
Copy Markdown
Contributor

ov::runtime::Tensor() -> {}

no, in this case we can call read_model(string, string) which is wrong
Got it

@ilya-lavrenov ilya-lavrenov merged commit 972524f into openvinotoolkit:master Oct 13, 2021
@ilyachur ilyachur deleted the fix_read_model_api branch October 13, 2021 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants