perf(qwen3.5-fp4-b200-sglang-mtp): add TEP2 arm on NVFP4-V2 / perf(qwen3.5-fp4-b200-sglang-mtp): 基于 NVFP4-V2 添加 TEP2 组合 - #2550
Conversation
Add a TP2/EP2 arm to the Qwen3.5-397B-A17B NVFP4 B200 SGLang MTP 8k/1k sweep at concurrency 16/32/64, alongside the existing TP2/EP1 rows.
|
Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase For PR verification, add the PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs 感谢你的贡献!请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人,你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板,包括保留英文语句 如需进行 PR 验证,请为此 PR 添加 PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档 |
There was a problem hiding this comment.
LGTM — straightforward benchmark config addition.
Extended reasoning...
Overview
This PR adds a single new search-space arm (tp:2, ep:2, conc-list:[16,32,64], spec-decoding:mtp) to the existing qwen3.5-fp4-b200-sglang-mtp sweep in configs/nvidia-master.yaml, and appends a corresponding perf-changelog.yaml entry documenting the measured gains (+5-9% tok/s/GPU and tok/s/user across concurrencies 16/32/64) and correctness checks (GSM8K strict-match/flexible-extract above threshold, coherence pass, MTP acceptance baseline-like).
Security risks
None. This is a pure benchmark configuration change — no code paths, auth, or data handling are touched.
Level of scrutiny
Low. This follows the exact same mechanical pattern used throughout this file for adding sweep arms (confirmed conc-list syntax matches other entries in the config), and the changelog entry follows the established schema with the required fields (config-keys, scenario-type, description, pr-link) matching the format of adjacent entries.
Other factors
No outstanding review comments in the timeline beyond the standard automated recipe-reminder bot message. The perf-changelog description includes specific before/after benchmark numbers and correctness validation results, consistent with prior similar TEP-arm-addition PRs in the repo history.
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=31432304739 |
|
As a PR reviewer and CODEOWNER, I have reviewed this and have:
Additional detail section:Scope of the change. This PR adds a single search-space row to the existing Validation and eval evidence. Run 31432304739 ran on the exact PR head Speculative decoding and chat template. The new arm is a real configuration, not a relabel. The script consumes expert parallelism directly — Model and scenario scope. MODELS.md lists Qwen3.5-397B-A17B as active for Single-turn 8k1k limited to fp8/fp4, and records the deprecations as Single-turn 1k1k, Single-turn 1k8k, and all bf16 recipes. This submission is 8k1k NVFP4 and adds no 1k1k lane. MODELS.md also records the engine expectation for this model as the native/upstream SGLang engine with native MTP, which is exactly what this arm exercises on the upstream Single-node recipe publication — left unchecked, needs a scope call. This is a single-node SGLang submission, so the item is at least arguably in scope, and no upstream recipe or cookbook link has been provided on this PR. Against that: the PR introduces no recipe file and no new serving script — it extends an already-published single-node config's search space with a parallelism variant, and the underlying No engine or serving-stack patching. The diff is two YAML files. No Signed: |
❌❌❌ REJECTED ❌❌❌@Ankur-singh Blocking issue: this is a single-node SGLang submission, but no merged upstream recipe PR or published cookbook page is linked in the sign-off (the recipe checklist item is unchecked). The new ✅ Check 0 (CODEOWNER): PASS — |
# Conflicts: # perf-changelog.yaml
# Conflicts: # perf-changelog.yaml
# Conflicts: # perf-changelog.yaml
精简 perf-changelog 条目:仅保留 TEP2 组合与 #2205 NVFP4-V2 依赖两条说明。
将 origin/main 合并入本分支,并按主干条目 + 本分支尾部条目的方式解决 perf-changelog 冲突。
…cription 合并远端分支上 Klaud 自动生成的 main 合并提交,perf-changelog 条目保留精简后的两条说明。
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=31625807465 |
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=31628689774 |
|
/reuse-sweep-run 31628689774 |
|
As a PR reviewer and CODEOWNER, I have reviewed this and have:
Additional detail section:Scope of the change. One search-space row added to the existing Validation and eval evidence. Run 31628689774 executed on the exact current PR head Checkpoint provenance — why the earlier run was not used. An earlier green sweep on this PR (run 31432304739) measured Speculative decoding and chat template. The new arm is a real configuration, not a relabel. The script consumes expert parallelism directly — Model and scenario scope. MODELS.md keeps fp8 and fp4 recipes for Qwen3.5-397B-A17B active and deprecates only the bf16 recipes; the speculative-decoding A/B retirements listed there concern agentic coding, not Single-turn 8k1k. This submission is 8k1k NVFP4 and adds no deprecated lane. Single-node recipe publication. Documented upstream in the SGLang cookbook via sgl-project/sglang#34357, MERGED 2026-08-11. It records B200 NVFP4 as No engine or serving-stack patching. The diff is two YAML files. No Merge state. This sign-off attests to review and validation only. The PR currently conflicts with main and must be brought up to date before merge; that is a separate merge-time step and does not affect the evidence above. Signed: |
✅✅✅ Verdict: PASS ✅✅✅✅ Check 0 (CODEOWNER): PASS — signer |
|
/reuse-sweep-run |
# Conflicts: # perf-changelog.yaml
✅✅✅ Verdict: PASS ✅✅✅✅ Check 0 (CODEOWNER): PASS — |
Summary
Add a TEP2 (tp 2, ep 2) arm to the
qwen3.5-fp4-b200-sglang-mtp8k/1k fixed-sequence-length MTP sweep at concurrency 16, 32, and 64. This arm is measured on thenvidia/Qwen3.5-397B-A17B-NVFP4-V2checkpoint that landed via #2205; no other configs are touched.Scope:
qwen3.5-fp4-b200-sglang-mtplmsysorg/sglang:v0.5.14-cu130nvidia/Qwen3.5-397B-A17B-NVFP4-V2(inherited from Switch Qwen3.5 FP4 SGLang configs to NVFP4 V2 / 将 Qwen3.5 FP4 SGLang 配置切换到 NVFP4 V2 #2205, resolved viarunners/launch_b200-dgxc.sh)--use-chat-templateChanges
configs/nvidia-master.yaml: append{ tp: 2, ep: 2, conc-list: [16, 32, 64], spec-decoding: mtp }to theqwen3.5-fp4-b200-sglang-mtp8k/1ksearch-space.perf-changelog.yaml: append the corresponding entry, calling out that the arm rides on the NVFP4-V2 checkpoint switch from Switch Qwen3.5 FP4 SGLang configs to NVFP4 V2 / 将 Qwen3.5 FP4 SGLang 配置切换到 NVFP4 V2 #2205, with the PR link backfilled.Results
Numbers are from the exact-head Run Sweep on
qwen3.5-fp4-b200-sglang-mtpand compared against the current publishedtp 2, ep 1rows onmain:tp 2, ep 1reference at concurrency 32: +1.11% tok/s/GPU and +2.28% tok/s/user, non-overlapping on both axes across two samples.tp 2, ep 1rows onmain:Notes
NVFP4-V2onb200-dgxcare already merged and require no further edits here.中文说明
概要
向
qwen3.5-fp4-b200-sglang-mtp的 8k/1k 定长序列 MTP 扫描中新增一个 TEP2(tp 2, ep 2)组合,覆盖并发度 16、32、64。本 PR 基于 #2205 落地的nvidia/Qwen3.5-397B-A17B-NVFP4-V2检查点进行测量;不涉及其他配置。范围:
qwen3.5-fp4-b200-sglang-mtplmsysorg/sglang:v0.5.14-cu130nvidia/Qwen3.5-397B-A17B-NVFP4-V2(继承自 Switch Qwen3.5 FP4 SGLang configs to NVFP4 V2 / 将 Qwen3.5 FP4 SGLang 配置切换到 NVFP4 V2 #2205,通过runners/launch_b200-dgxc.sh路由)--use-chat-template变更内容
configs/nvidia-master.yaml:向qwen3.5-fp4-b200-sglang-mtp8k/1k 的search-space追加{ tp: 2, ep: 2, conc-list: [16, 32, 64], spec-decoding: mtp }。perf-changelog.yaml:在文件末尾追加对应条目,并注明该组合建立在 Switch Qwen3.5 FP4 SGLang configs to NVFP4 V2 / 将 Qwen3.5 FP4 SGLang 配置切换到 NVFP4 V2 #2205 切换到 NVFP4-V2 检查点之后,PR 链接已回填。结果
数据来自
qwen3.5-fp4-b200-sglang-mtp对应 head 的 Run Sweep,与main上当前发布的tp 2, ep 1结果对比:tp 2, ep 1参考:tok/s/GPU +1.11%,tok/s/user +2.28%,两次采样在两个轴上均不重叠。main上当前发布的tp 2, ep 1行:备注
b200-dgxc上NVFP4-V2的模型、启动脚本与 squash 路径已合并,此处无需额外改动。