Code Review

Author: winskuo-quic

.ci/scripts/test_model.sh

@@ -176,8 +176,6 @@ test_model_with_qnn() {
   export LD_LIBRARY_PATH=$QNN_SDK_ROOT/lib/x86_64-linux-clang/
   export PYTHONPATH=$EXECUTORCH_ROOT/..
 
-  pip install pydot
-
   EXTRA_FLAGS=""
   # Ordered by the folder name, then alphabetically by the model name
   # Following models are inside examples/qualcomm/scripts folder

.ci/scripts/test_qnn_static_llm.sh

@@ -35,7 +35,6 @@ fi
 which "${PYTHON_EXECUTABLE}"
 
 # Although static llama CI does not require graphviz, it is required by test_qnn_delegate.py
-pip install pydot
 pip install graphviz
 
 set +e

backends/qualcomm/scripts/build.sh

@@ -6,6 +6,8 @@
 # LICENSE file in the root directory of this source tree.
 set -e
 
+pip install pydot
+
 # Check if running on macOS/Darwin
 if [[ "$(uname -s)" == "Darwin" ]]; then
     echo "Error: Qualcomm backend Python interface requires Linux operating system."