[GitHub] [tvm] manupa-arm commented on a change in pull request #9323: Fix Arm(R) Ethos(TM)-U55 NPU Demo app

[GitBox <gi...@apache.org>]

manupa-arm commented on a change in pull request #9323:
URL: https://github.com/apache/tvm/pull/9323#discussion_r734821111



##########
File path: tests/scripts/task_cpp_unittest.sh
##########
@@ -45,3 +45,11 @@ cd apps/bundle_deploy
 rm -rf build
 make test_dynamic test_static
 cd ../..
+
+# Test Arm(R) Cortex(R)-M55 CPU and Ethos(TM)-U55 NPU demo app
+if test -f "/opt/arm/FVP_Corstone_SSE-300_Ethos-U55/models/Linux64_GCC-6.4/FVP_Corstone_SSE-300_Ethos-U55" && pip3 list | grep vela; then
+    sudo pip3 install -e python

Review comment:
       Im fine with either, we can put a comment in the script to say that (one could use tvmc directly once Pip installed) or we can move it to bit higher position in terms of scripts.
   
   Since this tested now, if we need more time to figure out where to put the installation, happy to take python3 -m tvm.driver.tvmc (with a comment explaining how it could be used alternatively)




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