-
Notifications
You must be signed in to change notification settings - Fork 445
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
[OTE] Update OpenVINO to 2022.3 release #1393
Conversation
sovrasov
commented
Nov 25, 2022
•
edited
Loading
edited
- Update OV
- Update NNCF, corresponding fixes in d-o-r
- Minor fixes in readmes
- Changes from [DEV] Fix nncf config for padim model #1487
run ote-test |
2dfa92c
to
6f44cb1
Compare
run ote-test |
1 similar comment
run ote-test |
d2a8a3c
to
671c62d
Compare
run ote-test |
1 similar comment
run ote-test |
671c62d
to
641a10a
Compare
run ote-test |
run ote-test |
4 similar comments
run ote-test |
run ote-test |
run ote-test |
run ote-test |
run ote-test |
run ote-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@sovrasov, @goodsong81, do you guys know why we need to install both openvino and openvino-dev? Is there any specific motivation to have this installation? Can't we just do pip install openvino["dev"] that would install the dev package as well as the base openvino ? Or why openvino-dev does not install the base openvino? Any ideas, thoughts? |
Hi Samet, you're right, we can omit openvino if we have openvino-dev. openvino package version is always presented here for clarity and historically we always added it implicitly. Perhaps, we may change that in otx. |
Vlad, thanks for the clarification!
To be honest, it caused more confusion to me than clarity. I would agree to only keep |