Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -173,5 +173,6 @@ if [ "${EVAL_ONLY}" = "true" ]; then
run_eval --port "$PORT"
else
build_replay_cmd "$RESULT_DIR"
REPLAY_CMD+=" --allow-empty-content"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AIPerf empty-content flag unpinned

High Severity

The recipe now passes --allow-empty-content to AIPerf, but this commit does not pin utils/aiperf to e8ece5d29bd0285af1ced36fbc4c670328b2d58c. install_agentic_deps installs the local submodule, and current AIPerf does not accept that flag, so the MiniMax TRT AgentX replay will fail at CLI parse before TTFT can be measured.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit ef9707f. Configure here.

run_agentic_replay_and_write_outputs "$RESULT_DIR"
fi
9 changes: 9 additions & 0 deletions perf-changelog.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6225,3 +6225,12 @@
- "Add a TP2/EP2 HiCache branch and update the resident recipes with extra_buffer_lazy, ReplaySSM speculative verification, and checkpoint page-cache release."
- "Select TP4 C1/C2/C4/C8/C12/C20/C24, TP2/EP2 resident C4/C8/C12/C20, and TP2/EP2 HiCache C22/C24/C28/C32/C40 with golden synthetic acceptance length 3.39."
pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/2660

- config-keys:
- minimaxm3-fp4-b300-trtllm-agentic-mtp
scenario-type:
- agentic-coding
description:
- "Test AIPerf PR #42 empty-content TTFT handling on the MiniMax-M3 NVFP4 B300 TensorRT-LLM AgentX configuration."
- "Pin utils/aiperf to e8ece5d29bd0285af1ced36fbc4c670328b2d58c and enable --allow-empty-content for this recipe."
pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/2682
2 changes: 1 addition & 1 deletion utils/aiperf
Loading