You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/06/23 11:55:59 UTC

[GitHub] [tvm] hyongtao opened a new pull request #8311: docker-test-pr

hyongtao opened a new pull request #8311:
URL: https://github.com/apache/tvm/pull/8311


   Author: Yongtao Huang <hy...@vmware.com>
   Date:   Wed Jun 23 19:44:08 2021 +0800
   
       The dockerfiles are apply to deploy and test TVM environments (including CPU and NVIDIA_GPU) easily.
       Long log:
       - There are some errors when we re-build official dockerfile like demo_cpu: https://discuss.tvm.apache.org/t/error-about-tvm-docker/10301
       - I have build CPU_TVM and NVIDIA_GPU_TVM docker, which includes inference envs and code.
       - This can help us verify and deploy our ML model directly.
       Signed-off-by: Yongtao Huang <hy...@vmware.com>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] jroesch commented on pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
jroesch commented on pull request #8311:
URL: https://github.com/apache/tvm/pull/8311#issuecomment-1016830726


   This PR appears to be out of date, please feel free to reopen it if this is not the case.
   
   As part of the new year we are attempting to triage the project's open pull requests to ensure that code which
   is ready for review and/or merging receives adequate attention.
   
   Thanks again for your contribution, and feel free to reach out to discuss these changes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] areusch commented on a change in pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
areusch commented on a change in pull request #8311:
URL: https://github.com/apache/tvm/pull/8311#discussion_r667155913



##########
File path: docker/test/Dockerfile.CPU
##########
@@ -0,0 +1,37 @@
+FROM ubuntu:18.04
+
+ENV http_proxy $HTTP_PROXY
+ENV https_proxy $HTTPS_PROXY
+
+# 1. Prepare dependence.
+RUN apt-get update && apt-get install -y wget git vim python3 python3-dev python3-setuptools gcc libtinfo-dev zlib1g-dev build-essential cmake libedit-dev libxml2-dev
+RUN echo deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-9 main\
+     >> /etc/apt/sources.list.d/llvm.list
+RUN echo deb-src http://apt.llvm.org/bionic/ llvm-toolchain-bionic-9 main\
+     >> /etc/apt/sources.list.d/llvm.list
+RUN echo deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic main\
+     >> /etc/apt/sources.list.d/llvm.list
+RUN echo deb-src http://apt.llvm.org/bionic/ llvm-toolchain-bionic main\
+     >> /etc/apt/sources.list.d/llvm.list
+RUN wget -q -O - http://apt.llvm.org/llvm-snapshot.gpg.key| apt-key add -
+RUN apt-get update && apt-get install -y llvm-9 clang-9 libclang-9-dev
+
+# 2. Install TVM
+WORKDIR /workspace/
+RUN git clone --recursive https://github.com/apache/tvm tvm
+WORKDIR /workspace/tvm/
+RUN mkdir build && cp cmake/config.cmake build
+RUN cd build && echo set\(USE_LLVM llvm-config-9\) >> config.cmake && cmake .. && make -j4
+
+# 3. Prepare Python sitpackage
+RUN apt-get install -y python3-pip && pip3 install --upgrade pip
+RUN pip3 install numpy scipy decorator attrs tornado psutil tensorflow pytest

Review comment:
       i'm not sure things are going to work super well if you just install latest




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] hogepodge commented on pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
hogepodge commented on pull request #8311:
URL: https://github.com/apache/tvm/pull/8311#issuecomment-877293036


   @comaniac @hyongtao I'm on PTO until Monday, but will review the builds once I'm back at the office on Monday. I made some line edits to the Docker files. I would like to see more documentation on how to use the added test files. I'm also curious to know why they are docker specific tests and not more general integration tests.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] masahi commented on pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
masahi commented on pull request #8311:
URL: https://github.com/apache/tvm/pull/8311#issuecomment-1008436601


   @hyongtao Can you update?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] jroesch closed pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
jroesch closed pull request #8311:
URL: https://github.com/apache/tvm/pull/8311


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] comaniac commented on pull request #8311: docker-test-pr

Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #8311:
URL: https://github.com/apache/tvm/pull/8311#issuecomment-877000118


   @areusch @hogepodge could you help review this PR as you guys are working on docker related features recently?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org