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



##########
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:
       yeah the thing i wanted to point out is that when there is a PYTHONPATH 
override, the venv may not be effective. you'll at least need to remove that 
override. 
   
   if you go this route (venv + explicitly clear PYTHONPATH for this one test), 
can we file a cleanup issue to run all of the pytests in a venv with TVM 
properly installed using pip?




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