areusch commented on a change in pull request #10003:
URL: https://github.com/apache/tvm/pull/10003#discussion_r789176197



##########
File path: tests/scripts/task_ci_setup.sh
##########
@@ -39,3 +39,6 @@ python3 -m pip install --user tlcpack-sphinx-addon==0.2.1 
synr==0.6.0
 
 # Ensure no stale pytest-results remain from a previous test run.
 (cd build && rm -rf pytest-results)
+
+# Install tvm as a local package to allow tvmc to work
+python3 -m pip install --user -e ./python

Review comment:
       i think this would install tvm in `.local`, i wonder if it would cause 
some staleness issues for folks running this script locally and invoking 
`import tvm` (e.g. would it copy all the .pyc files into 
`~/.local/lib/pythonXX/tvm`)? maybe a better strategy is to `python3 
-mtvm.driver.tvmc` in `run_demo.sh` above, and just comment that this way is 
used to avoid these problems?




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to