Mousius commented on a change in pull request #9323:
URL: https://github.com/apache/tvm/pull/9323#discussion_r734801979
##########
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:
I'm not suggesting we shouldn't do it but this scripts responsibility is
to run unit tests, this installs TVM python and all its dependencies as a side
effect which I wouldn't expect from such a script.
--
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]
