comaniac commented on a change in pull request #6342:
URL: https://github.com/apache/incubator-tvm/pull/6342#discussion_r478609685
##########
File path: docker/Dockerfile.ci_cpu
##########
@@ -83,3 +83,7 @@ RUN bash /install/ubuntu_install_caffe.sh
# Github Arm(R) Ethos(TM)-N NPU driver
COPY install/ubuntu_install_ethosn_driver_stack.sh
/install/ubuntu_install_ethosn_driver_stack.sh
RUN bash /install/ubuntu_install_ethosn_driver_stack.sh
+
+# Vitis-AI PyXIR CI deps
+COPY install/ubuntu_install_vai_packages.sh
/install/ubuntu_install_vai_packages.sh
+RUN bash /install/ubuntu_install_vai_packages.sh
Review comment:
I agree with Leandro basically. In TVM we have two types of docker
files: CI and Demo. Demo is used to provide an environment for users to quickly
try out basic TVM functions. However, TVM does not provide the docker file for
developments other than CI environment.
If you would like to provide an environment for TVM users that want to use
Vitis-AI, the currently solution is either providing installation instructions
in the tutorial or document like ACL does to help users set up the environment,
or make it as a Demo image and point users to there.
Of course, ideally it would be better for TVM to have a set of docker files
for developments, but it is hard to achieve for some backends due to the
license issue.
cc @tqchen for more comments.
----------------------------------------------------------------
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:
[email protected]
