Fix QNN llama script Python path#20751
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20751
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New Failures, 1 Cancelled Job, 1 Unrelated FailureAs of commit d44c8aa with merge base 6edd49d ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
BROKEN TRUNK - The following job failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
Summary:
Fix QNN llama CI scripts so they resolve EXECUTORCH_ROOT to the repository root and set PYTHONPATH to the parent of that root. The Qualcomm examples are source-tree modules, so the CI import path needs the repo parent instead of a cwd-relative '..'. This addresses failures such as:
python: No module named executorch.examples.qualcomm.oss_scripts.llama.eval_llama_qnn
Test Plan:
bash -n .ci/scripts/test_llama.sh .ci/scripts/test_qnn_static_llama_eval.sh .ci/scripts/test_qnn_static_llm.sh
git diff --check origin/main
lintrunner -r origin/main
Verified the new EXECUTORCH_ROOT expression resolves to the repo root for all three scripts.
Authored with Claude.