Skip to content

Commit

Permalink
If benchmark fails in CI, try again without the --dd flag, to see if …
Browse files Browse the repository at this point in the history
…we're just missing a GAMS license.
  • Loading branch information
SamRWest committed Feb 14, 2024
1 parent 52ae6b5 commit 4572d0e
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,9 @@ jobs:
# printed again in the next step.
# Save the return code to retcode.txt so that the next step can fail the action
# if run_benchmarks.py failed.
#
# If it fails, try again without the --dd flag, to see if we're just missing a GAMS license.
# Build still fails without GAMS in final step, but we get CSV-only regression test results here - useful for testing in repo forks.
run: |
source .venv/bin/activate
export PATH=$PATH:$GITHUB_WORKSPACE/GAMS/gams44.1_linux_x64_64_sfx
Expand All @@ -100,6 +103,14 @@ jobs:
--verbose \
| tee out.txt; \
echo ${PIPESTATUS[0]} > retcode.txt)
if [ $(cat retcode.txt) -ne 0 ]; then
echo "Retrying without GAMS to check CSV regressions..."
(python utils/run_benchmarks.py benchmarks.yml \
--times_dir $GITHUB_WORKSPACE/TIMES_model \
--verbose \
| tee out.txt\
echo ${PIPESTATUS[0]} > retcode.txt)
fi
- name: Print summary
working-directory: xl2times
Expand Down

0 comments on commit 4572d0e

Please sign in to comment.