Skip to content

Commit

Permalink
don't double-install executorch for one-off linux pull-time jobs (pyt…
Browse files Browse the repository at this point in the history
…orch#8142)

I remember somebody mentioning they wanted install_executorch to be the one solution previously. CI is red for these jobs due to timeouts and we should not block a time improvement just because it is not the ideal long-term solution.
  • Loading branch information
swolchok authored Feb 4, 2025
1 parent 9b3f2ba commit 81f7c4f
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions .github/workflows/pull.yml
Original file line number Diff line number Diff line change
Expand Up @@ -220,9 +220,6 @@ jobs:
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-linux.sh "cmake"
# install pybind
bash install_executorch.sh --pybind xnnpack --use-pt-pinned-commit
# install Llava requirements
bash examples/models/llama/install_requirements.sh
bash examples/models/llava/install_requirements.sh
Expand Down Expand Up @@ -483,9 +480,6 @@ jobs:
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-linux.sh "cmake"
# install pybind
bash install_executorch.sh --pybind xnnpack --use-pt-pinned-commit
# install phi-3-mini requirements
bash examples/models/phi-3-mini/install_requirements.sh
Expand Down Expand Up @@ -513,9 +507,6 @@ jobs:
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-linux.sh "cmake"
# install pybind
bash install_executorch.sh --pybind xnnpack --use-pt-pinned-commit
# install llama requirements
bash examples/models/llama/install_requirements.sh
Expand Down Expand Up @@ -543,9 +534,6 @@ jobs:
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-linux.sh "cmake"
# install pybind
bash install_executorch.sh --pybind xnnpack --use-pt-pinned-commit
# install llama requirements
bash examples/models/llama/install_requirements.sh
Expand Down Expand Up @@ -573,9 +561,6 @@ jobs:
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-linux.sh "cmake"
# install pybind
bash install_executorch.sh --pybind xnnpack --use-pt-pinned-commit
# install llama requirements
bash examples/models/llama/install_requirements.sh
Expand Down

0 comments on commit 81f7c4f

Please sign in to comment.