Repository navigation
[CI/Build] Refresh A3 and MiniCPM-o H100 performance baselines - #6538
Conversation
Align A3, MiniMax-H3, and Qwen3-Omni thresholds with the agreed August nightly windows. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
This PR was classified as CI work. CI owner: @yenuo26 @congw729, please review your own changes and leave a short self-review comment describing what you checked. PRs without author self-review may not be assigned a reviewer. Please take a look when you have a chance. If you would like an automated review, mention @vllm-omni-review-bot in a comment. |
Enable baseline assertions for A3 nightly coverage using the available August 6–13 results. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
e00e3a1 to
c5ffd91
Compare
Rebuild A3-only baselines on current main using the Sep 13-19 archived NPU nightly runs. Refresh 59 values in 8 files; exclude obsolete workload backends and cases with fewer than five comparable run days. Remove the previous H100 changes and superseded async/duplex baselines. Keep original PR history while incorporating upstream main. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
Omni ReviewBot triage noteAutomated triage of commit
These are automated triage suggestions only — the final decision belongs to the maintainers. |
Recompute the five concurrency-8 baselines from the Sep 18 and 19 successful chat runs. Exclude the older latency regime and the temporary Sep 16-17 realtime backend. Use the selected two-night window for this case; retain the existing sampling windows for other A3 configurations. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
|
@yenuo26 Can be merged |
| 1857.2154, | ||
| 6600.5101, | ||
| 10450.6811 | ||
| 2457.6655, |
There was a problem hiding this comment.
Does this need to be adjusted to ascending?
There was a problem hiding this comment.
This has been updated since your review. After checking the daily history, C1 E2E was revised from 2457.6655 to 2086.7028 ms, using the equal-weight mean of all six successful Aug 20–25 nights. This avoids pooling the later high-latency period, with no extra margin added. The PR description now includes the sampling window and all six raw results.
AI assistance: Codex helped verify the values and draft this reply.
| 0.5383, | ||
| 0.6042, | ||
| 0.7547 | ||
| 0.41, |
There was a problem hiding this comment.
Does this need to be adjusted to descending?
There was a problem hiding this comment.
This has also been updated since your review. C1 QPS was revised from 0.4100 to 0.4813, using the same Aug 20–25 six-night mean as C1 E2E. All six successful nights are included, with no extra margin added. It remains below the original 0.5383 baseline, and the selected historical data and calculation are documented in the PR description.
AI assistance: Codex helped verify the values and draft this reply.
|
@congw729 please resolve conflicts |
…ation Merge current main and rebuild Wan2.2 A3 baseline blocks on the new vllm bench serve --omni configuration introduced by vllm-project#7737. Rename throughput and memory metrics; convert the archived latency means from seconds to milliseconds before rounding to four decimals. Retain the selected nightly windows, including MiniCPM-o c8 Sep 18-19. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
Use all six successful Aug 20-25 nightly runs for C1 request throughput, E2E latency, audio first-packet latency and RTF, plus C8 audio first-packet latency. Keep equal nightly weights and round once to four decimals. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
Merge current main and rebuild the Qwen3-Omni baseline file from its updated configuration. Keep the random workload baseline empty after vllm-project#8057; the five archived A3 RTF values no longer describe that workload. Preserve all remaining 54 A3 values, including the five approved MiniCPM-o Aug 20-25 means. Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
|
@yenuo26 Could you please take another look? The five MiniCPM-o A3 baselines now use the Aug 20–25 six-night means, and the PR description and source links are updated. Conflicts with main are resolved; raw-data recomputation, pre-commit and CI checks passed. Thanks! |
Based on PR vllm-project#6538 head 9525cb9. Preserve its A3 baseline changes and update only the 15 H100 values for C1/P32, C4/P64 and C8/P128. Use all seven ordinary HTTP Seed-TTS runs per concurrency dated 2026-09-22 through 2026-09-28. Average full-precision nightly values with equal weights, then round once to four decimals without adding a margin. Exclude duplex_seed_tts because its timing differs. Record the window and reference-audio boundary in the existing description field. Source SHA-256: c1cc381debb1e55e1683620488fe0b89c2a22acbaa21d3e46d4b6f35a9599646 Validation: 21 unique archived runs; 1568 completed requests with zero reported failures and complete audio metric counts. All 15 values independently recomputed with Decimal, configuration scope checked, and targeted pre-commit passed. No new inference or WER run. Signed-off-by: natureofnature <wzliu@connect.hku.hk>
Based on PR vllm-project#6538 head 9525cb9. Preserve its A3 baseline changes and update only the 15 H100 values for C1/P32, C4/P64 and C8/P128. Use all seven ordinary HTTP Seed-TTS runs per concurrency dated 2026-09-22 through 2026-09-28. Average full-precision nightly values with equal weights, then round once to four decimals without adding a margin. Exclude duplex_seed_tts because its timing differs. Record the window and reference-audio boundary in the existing description field. Source SHA-256: c1cc381debb1e55e1683620488fe0b89c2a22acbaa21d3e46d4b6f35a9599646 Validation: 21 unique archived runs; 1568 completed requests with zero reported failures and complete audio metric counts. All 15 values independently recomputed with Decimal, configuration scope checked, and targeted pre-commit passed. No new inference or WER run. Signed-off-by: natureofnature <wzliu@connect.hku.hk>
|
I verified all 69 changed baseline values against the dashboard data: 54 A3 values and 15 H100 values match the stated nightly averages. The benchmark parameters are unchanged. The A3 sampling windows, including the historical references and limited two-night C8 coverage, are explicitly documented. Before merging, please address update the PR description to include the 15 H100 changes and remove the statement that GPU baselines are unchanged |
Based on PR vllm-project#6538 head 9525cb9. Preserve its A3 baseline changes and update only the 15 H100 values for C1/P32, C4/P64 and C8/P128. Use all seven ordinary HTTP Seed-TTS runs per concurrency dated 2026-09-22 through 2026-09-28. Average full-precision nightly values with equal weights, then round once to four decimals without adding a margin. Exclude duplex_seed_tts because its timing differs. Record the window and reference-audio boundary in the existing description field. Source SHA-256: c1cc381debb1e55e1683620488fe0b89c2a22acbaa21d3e46d4b6f35a9599646 Validation: 21 unique archived runs; 1568 completed requests with zero reported failures and complete audio metric counts. All 15 values independently recomputed with Decimal, configuration scope checked, and targeted pre-commit passed. No new inference or WER run. Signed-off-by: natureofnature <wzliu@connect.hku.hk> Signed-off-by: Alicia <115451386+congw729@users.noreply.github.com>
70e705d to
08aaa48
Compare
|
The updated description now covers both the A3 and H100 changes, and DCO passes. All 69 baseline values match the recomputed dashboard averages. One small documentation correction: please restore the explicit sampling window for MiniCPM-o A3 C1 TTFT and all C4 metrics: Sep 13–15 and Sep 18–19, five nights total. These do not use the full Sep 13–19 default window. For example, C1 TTFT averages 1091.2690 ms across all seven nights, versus the configured 328.1820 ms across the selected five. No additional code or numerical issues found. This verification covers the archived dashboard data, not a new inference run. Once the sampling note is restored and the remaining CI checks pass, this looks ready to merge. |
|
@natureofnature Restored the explicit Sep 13–15 and Sep 18–19 (five nights) window for MiniCPM-o A3 C1 TTFT and all C4 metrics in the PR description. It also explains that Sep 16–17 are excluded because they used the temporary realtime backend. All baseline values remain unchanged. AI assistance: Codex assisted with this documentation correction and reply. |
Summary
Refresh 69 performance baseline values across 8 files: 54 A3 values and 15 MiniCPM-o H100 values.
All values use equal-weight nightly arithmetic means, rounded once to four decimals, without an additional margin. Benchmark parameters, deployment settings and regression tolerances are unchanged.
MiniCPM-o baseline increases
A3 MiniCPM-o: historical comparison and window selection
The August–September audit, covering successful standard HTTP Seed-TTS runs through Sep 23, identified distinct performance periods. Values below are equal-weight nightly means; latency is in milliseconds.
The selected window balances historical coverage and regression sensitivity:
These values serve as historical performance references. The comparison supports the chosen sampling window; the causes of subsequent performance shifts remain incompletely explained.
H100: the stored baselines predate #6628, which enabled the HTTP path to use each request’s reference audio. This changes the conditioning workload and supports refreshing the reference, although the available results do not isolate its contribution to each performance difference.
The largest increases are:
The full seven-day window includes performance improvements during the week. C8 audio TTFP fell from approximately 2.99–3.09 seconds on Sep 22–24 to 2.04–2.20 seconds on Sep 25–28. Its seven-night baseline is therefore looser than the later observations and will be less sensitive to regressions from that improved level. No slower days were selectively removed.
These archived means describe the selected windows. Exact software revisions are incomplete for the H100 records, so they do not establish a fixed-version performance comparison or fully explain the increases. The C8 RTF < 1 optimization goal remains outstanding.
Validation and sources
git diff --checkpassed. No new inference or WER runs.Sources: A3 September archive, A3 August archive, H100 archive, and H100 update details.
Qwen3-Omni random-workload baselines remain omitted after #8057. Wan2.2 retains the archived means with the metric-name and latency-unit conversion from #7737.
AI assistance: Codex assisted with archive analysis, baseline verification and PR wording.