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.

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

Apa/hetero no legacy #3398

Merged
merged 3 commits into from
Nov 30, 2020

Conversation

ilya-lavrenov
Copy link
Contributor

Resolved merge conflicts for #2996

@ilya-lavrenov ilya-lavrenov added the category: HETERO OpenVINO HETERO plugin label Nov 27, 2020
@ilya-lavrenov ilya-lavrenov added this to the 2021.3 milestone Nov 27, 2020
@openvino-pushbot openvino-pushbot added the category: inference OpenVINO Runtime library - Inference label Nov 27, 2020
@ilyachur ilyachur merged commit fb1b037 into openvinotoolkit:master Nov 30, 2020
@ilya-lavrenov ilya-lavrenov deleted the apa/hetero_no_legacy branch November 30, 2020 08:59
evolosen pushed a commit to evolosen/openvino that referenced this pull request Dec 3, 2020
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
@myshevts
Copy link
Contributor

myshevts commented Dec 3, 2020

Upon rebasing my private new fetaure branch to the latest master (that includes the MULTI switching to the ngraph only) , I have realized that "MULTI:CPU,GPU" now doesn't accept some old IR (ver4 that the MLS team is actively playing with) and this is fine, but CPU and GPU alone are still reading fine this old IR. This is a little bit of inconsistency. Generally the ngraph (and/or v10 IR ver) check should be done in some common place, maybe even IR parsing. Fyi, @ilya-lavrenov and @slyalin

@ilya-lavrenov
Copy link
Contributor Author

ilya-lavrenov commented Dec 3, 2020

@myshevts just remove inference_engine_ir_v7_reader from your build artifacts and you will get an exception you expected.
inference_engine_ir_v7_reader is used only for our tests (since still a lot of them are based on IR v7 and earlier), but it's not a part of OpenVINO release package.

mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 11, 2020
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 16, 2020
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Jan 14, 2021
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
jiwaszki pushed a commit to akuporos/openvino that referenced this pull request Jan 15, 2021
Co-authored-by: apankratovantonp <[email protected]>
Co-authored-by: Alexander Zhogov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: HETERO OpenVINO HETERO plugin category: inference OpenVINO Runtime library - Inference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants