Skip to content

Commit 852a19c

Browse files
alec-flowersclaude
andcommitted
fix: direct warmup artifacts to ARTIFACT_DIR instead of cwd
The aiperf warmup call was missing --artifact-dir, causing it to write artifacts to the working directory (creating artifacts/ in repo root). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 07b70e9 commit 852a19c

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

  • src/srtctl/benchmarks/scripts/trace-replay

src/srtctl/benchmarks/scripts/trace-replay/bench.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,8 @@ python3 -c "import tiktoken" 2>/dev/null || pip install tiktoken
7878

7979
# Run small benchmark for warmup
8080
echo "Running warmup..."
81+
WARMUP_DIR="${ARTIFACT_DIR}/warmup"
82+
mkdir -p "${WARMUP_DIR}"
8183
aiperf profile \
8284
-m "${MODEL_NAME}" \
8385
--tokenizer "${TOKENIZER_PATH}" \
@@ -87,7 +89,8 @@ aiperf profile \
8789
--ui simple \
8890
--extra-inputs ignore_eos:true \
8991
--concurrency 1 \
90-
--request-count 5
92+
--request-count 5 \
93+
--artifact-dir "${WARMUP_DIR}"
9194
echo "Warmup complete"
9295

9396
# Setup artifact directory

0 commit comments

Comments
 (0)