Skip to content

Commit 07c4ab0

Browse files
sbryngelsonclaude
andcommitted
ci: fix fallback partition message, remove dead RETRY_CLEAN_CMD, fix precision flag
- run_parallel_benchmarks.sh: fix warning message to say gpu-rtx6000 (not gpu-l40s) to match the actual fallback value - phoenix/bench.sh: remove RETRY_CLEAN_CMD prefix on retry_build — retry-build.sh only uses RETRY_VALIDATE_CMD and always does rm -rf build on retry; variable was dead and re-introduced by upstream merge - test.yml: fix --${{ matrix.precision }} expanding to bare '--' when precision is empty; restore PRECISION env var pattern; remove manual 'Set up Python 3.14' step Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 38df383 commit 07c4ab0

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.github/scripts/run_parallel_benchmarks.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ if [ "$device" = "gpu" ] && [ "$cluster" = "phoenix" ]; then
3636
fi
3737
done
3838
if [ -z "$BENCH_GPU_PARTITION" ]; then
39-
echo "WARNING: No idle GPU partition found; falling back to gpu-l40s (may queue)"
39+
echo "WARNING: No idle GPU partition found; falling back to gpu-rtx6000 (may queue)"
4040
BENCH_GPU_PARTITION="gpu-rtx6000"
4141
fi
4242
export BENCH_GPU_PARTITION

.github/workflows/phoenix/bench.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ fi
2222
rm -rf build
2323

2424
source .github/scripts/retry-build.sh
25-
RETRY_CLEAN_CMD="./mfc.sh clean" retry_build ./mfc.sh build -j $n_jobs $build_opts || exit 1
25+
retry_build ./mfc.sh build -j $n_jobs $build_opts || exit 1
2626

2727
./mfc.sh bench $bench_opts -j $n_jobs -o "$job_slug.yaml" -- -c phoenix-bench $device_opts -n $n_ranks
2828

0 commit comments

Comments
 (0)