Skip to content
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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 3 commits into from
Oct 13, 2021
Merged

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

merged 3 commits into from
Oct 13, 2021

Conversation

ilyachur
Copy link
Contributor

@ilyachur ilyachur commented Oct 12, 2021 •
edited by ilya-lavrenov
Loading

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
@ilyachur ilyachur mentioned this pull request Oct 12, 2021
Closed
@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
apankratovantonp
apankratovantonp approved these changes Oct 12, 2021
View reviewed changes
Copy link
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
Contributor

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

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

@apankratovantonp
Copy link
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
Reviewers

@apankratovantonp apankratovantonp apankratovantonp approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

Assignees

@ilya-lavrenov ilya-lavrenov

Labels
category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference
Projects
None yet
Milestone
2022.1
Development

Successfully merging this pull request may close these issues.
4 participants
@ilyachur @ilya-lavrenov @apankratovantonp @openvino-pushbot