Skip to content

Commit 0e72c6f

Browse files
committed
Update on "Add preprocess to ci"
1. Run `test_preprocess.py` in ci. 2. Refactor `test_preprocess.py` by creating the model artifacts (eager, exported, et) once and re-use them on different inputs (previously, create model artifacts for each test). Test Plan: ``` pytest -c /dev/null -v -n auto examples/models/llama3_2_vision/preprocess/ ``` [ghstack-poisoned]
2 parents 1830375 + a52e1d5 commit 0e72c6f

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

.github/workflows/_unittest.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,9 @@ jobs:
7070
${CONDA_RUN} --no-capture-output \
7171
.ci/scripts/setup-macos.sh cmake
7272
73-
${PYTHON_EXECUTABLE} pip list
74-
7573
# Install llama3_2_vision dependencies.
76-
bash examples/models/llama3_2_vision/install_requirements.sh
74+
PYTHON_EXECUTABLE=python ${CONDA_RUN} --no-capture-output \
75+
./examples/models/llama3_2_vision/install_requirements.sh
7776
7877
# Run pytest with coverage
7978
${CONDA_RUN} pytest -n auto --cov=./ --cov-report=xml

0 commit comments

Comments
 (0)