hzfan commented on a change in pull request #15855: [Numpy] cumprod
URL: https://github.com/apache/incubator-mxnet/pull/15855#discussion_r326845669
 
 

 ##########
 File path: ci/docker/runtime_functions.sh
 ##########
 @@ -367,6 +367,7 @@ build_ubuntu_cpu_openblas() {
     set -ex
     export CC="gcc"
     export CXX="g++"
+    pip3 install --user psutil
 
 Review comment:
   > How about you add Ubuntu tvm to the CPU dockerfile?
   
   Yes, but I will also need a new file `ubuntu_tvm_cpu.sh` because 
`ubuntu_tvm.sh` installs tvm with cuda and cannot be run on cpu.
   
   > I mean if we want to run tvm cpu tests, we should also install it, right?
   
   No, it is not installed in ci. TVM is a 3rdparty in MXNet, so it is 
installed in cmake.
   
   Will a new file like `ubuntu_tvm_cpu.sh` be acceptable?

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to