Skip to content

Commit

Permalink
dump out pip requirements for repro, and add default FHT_branch
Browse files Browse the repository at this point in the history
  • Loading branch information
fabianlim committed May 20, 2024
1 parent 6fcbbfe commit 4fbc88b
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
7 changes: 7 additions & 0 deletions scripts/run_benchmarks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ DATA_CACHE=data/cache.json
# final result placed here
BENCH_RESULT_FILE=benchmarks.csv

# freeze the pip requirements here
PIP_REQUIREMENTS_FILE=requirements.txt

# env inputs
DRY_RUN=${DRY_RUN:-"false"}
NO_DATA_PROCESSING=${NO_DATA_PROCESSING:-"false"}
Expand Down Expand Up @@ -64,6 +67,7 @@ DEFAULTS_CONFIG=$WORKING_DIR/$DEFAULTS_CONFIG
ACCELERATE_CONFIG=$WORKING_DIR/$ACCELERATE_CONFIG
DATA_CACHE=$RESULT_DIR/$DATA_CACHE
BENCH_RESULT_FILE=$RESULT_DIR/$BENCH_RESULT_FILE
PIP_REQUIREMENTS_FILE=$RESULT_DIR/$PIP_REQUIREMENTS_FILE

# ------------- EXTRA ARGS -----------------

Expand All @@ -82,6 +86,9 @@ if [ "$NO_DATA_PROCESSING" = "true" ]; then
EXTRA_ARGS="$EXTRA_ARGS --no_data_processing"
fi

# dump out the environment
pip freeze > $PIP_REQUIREMENTS_FILE

# run the bench
python $WORKING_DIR/benchmark.py \
--num_gpus $NUM_GPUS_MATRIX \
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ commands =
# need a version of fms-hf-tuning that has integrated the framework
# NOTE: have to install this first coz havnt merged
# - this repo has a lot of pins, so we just install it first
pip install "fms-hf-tuning[flash-attn] @ git+https://github.com/fabianlim/fms-hf-tuning.git@"{env:FHT_BRANCH}
pip install "fms-hf-tuning[flash-attn] @ git+https://github.com/fabianlim/fms-hf-tuning.git@"{env:FHT_BRANCH:main}

# some models need this for tokenizers
pip install protobuf
Expand Down

0 comments on commit 4fbc88b

Please sign in to comment.