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

Use OpenVINO from pip #442

Closed
wants to merge 1 commit into from
Closed

Use OpenVINO from pip #442

wants to merge 1 commit into from

Conversation

dkurt
Copy link

@dkurt dkurt commented Apr 16, 2021

test run: https://github.com/dkurt/deepvariant/actions/runs/755874669

@pichuan
Copy link
Collaborator

pichuan commented Apr 17, 2021

Thanks @dkurt !
I'll take a look next week and make sure I can incorporate the changes internally.
Can you confirm again that you're ok with us doing that? (I'll add a pointer to this PR when I make the internal change, so that the commit log will have the information.

Thanks.

@dkurt
Copy link
Author

dkurt commented Apr 17, 2021

@pichuan, Sure, I don't mind if you just copy the changes. Yes, this way is valid because OpenVINO from pip uses manylinux toolchain so it's compatible with Ubuntu 16.04 and allows to upgrade to newer versions.

@pichuan
Copy link
Collaborator

pichuan commented Apr 20, 2021

@dkurt FYI: This is now the merged internally and will come out in the next release.
Thanks!

@pichuan pichuan closed this Apr 20, 2021
@dkurt dkurt deleted the openvino_pip branch April 20, 2021 18:41
@pichuan
Copy link
Collaborator

pichuan commented May 29, 2021

@dkurt Question for you: I'm working an updates to update to Ubuntu20.04, TF2.5, Python3.8, etc.
Want to check in with you on whether we need to update any OpenVINO version before the next release.
Let me know! If it's easier through email, please email me at pichuan@google.com. Thanks!

@dkurt
Copy link
Author

dkurt commented May 31, 2021

Hi, @pichuan! OpenVINO 2021.3 is the latest version. 2021.4 will be released only at least in next month so it's OK to use 2021.3. Thanks!

@pichuan
Copy link
Collaborator

pichuan commented May 31, 2021

Thanks @dkurt ! 2021.3 sounds good. I won't make any changes now.

@pichuan
Copy link
Collaborator

pichuan commented Jun 2, 2021

@dkurt One more question - I'm comparing runtime on TF 2.5 and I have another question for you. Can you send me an email at at pichuan@google.com and we can discuss in more details? (I don't have your email address :))

pichuan added a commit that referenced this pull request Jul 29, 2021
## Use OpenVINO from pip

* Install OpenVINO by pip
* Update OpenVINO to latest 2021.3 version
* Use `enum34==1.1.8` to fix "AttributeError: module 'enum' has no attribute 'IntFlag'" (python-poetry/poetry#1122 (comment))

test run: https://github.com/dkurt/deepvariant/actions/runs/755874669

PiperOrigin-RevId: 369485495
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants