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

Updated Dockerfile for OpenvinoEP #1362

Merged
merged 8 commits into from
Jul 17, 2019
Merged

Conversation

avidiyal
Copy link
Contributor

@avidiyal avidiyal commented Jul 8, 2019

Signed-off-by: avidiyal [email protected]

Description:
-Updated Docker file for Openvino EP and also Readme.

Motivation and Context

This supports building the docker image for Openvino EP which supports accelerators CPU,GPU and VAD-R

@avidiyal avidiyal requested a review from a team as a code owner July 8, 2019 23:51
@msftclas
Copy link

msftclas commented Jul 8, 2019

CLA assistant check
All CLA requirements met.

Signed-off-by: avidiyal <[email protected]>
@jywu-msft
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 22 pipeline(s).

@jywu-msft
Copy link
Member

please fix the conflict. thanks.

@jywu-msft
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 22 pipeline(s).

@@ -120,3 +178,4 @@
```
curl -X POST -d "@request.json" -H "Content-Type: application/json" http://0.0.0.0:{your_local_port}/v1/models/mymodel/versions/3:predict
```
>>>>>>> master
Copy link
Member

Choose a reason for hiding this comment

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

why is this line here?

@jywu-msft
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 22 pipeline(s).

ENV PATH /opt/cmake/bin:$PATH
ENV LANG en_US.UTF-8
RUN wget https://github.com/Kitware/CMake/releases/download/v3.13.2/cmake-3.13.2-Linux-x86_64.tar.gz && \
tar -xf cmake-3.13.2-Linux-x86_64.tar.gz --strip 1 -C /opt/cmake
Copy link
Contributor

Choose a reason for hiding this comment

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

remove cmake..tar.gz after this step.

dockerfiles/README.md Show resolved Hide resolved

Retrieve your docker image in one of the following ways.

- For building the docker image, download OpenVINO online installer version 2018 R5.0.1 from [here](https://software.intel.com/en-us/openvino-toolkit/choose-download) and copy the openvino tar file in the same directory and build the image. The online installer size is only 16MB and the components needed for the accelerators are mentioned in the dockerfile.
Copy link
Member

Choose a reason for hiding this comment

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

I went to that link and couldn't download 2018 R5.0.1 , they only seem to have 2019 R1 available for download
Is there another direct link to R5? otherwise, this will be a problem.

Copy link
Member

Choose a reason for hiding this comment

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

I found https://download.01.org/openvinotoolkit/2018_R5/packages/
Is it officially supported?

Copy link
Member

Choose a reason for hiding this comment

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

the file I downloaded from there was l_openvino_toolkit_ie_p_2018.5.445.tgz
i don't think it works with the Dockerfile.openvino since it untars to a directory called inference_engine_vpu_arm and doesn't have some required files.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Attaching the screenshot for downloading the R5 version
image

Copy link
Member

Choose a reason for hiding this comment

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

R5 is not currently an option. I just signed up this morning. See screenshot.

Screenshot from 2019-07-11 12-43-33

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We are trying to resolve this issue. Will email as soon as it is fixed.

Copy link
Member

Choose a reason for hiding this comment

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

any updates on this issue?

dockerfiles/README.md Outdated Show resolved Hide resolved
@jywu-msft
Copy link
Member

jywu-msft commented Jul 15, 2019

the github CI notification issue was resolved so I can kick off the CI now.
but now the branch is out of date. can you update, please?

@avidiyal
Copy link
Contributor Author

Updated to the latest repo

@jywu-msft
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 22 pipeline(s).

@jywu-msft jywu-msft merged commit 859a57d into microsoft:master Jul 17, 2019
@suryasidd suryasidd deleted the updatedDocker_OVEP branch September 23, 2020 20:23
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.

6 participants