Skip to content

Retune DSV4 B300 AgentX MTP sweep / 调优 DSV4 B300 AgentX MTP 扫描 - #2536

Merged
cquil11 merged 6 commits into
mainfrom
agentx/dsv4-b300-mtp-upd
Aug 11, 2026
Merged

Retune DSV4 B300 AgentX MTP sweep / 调优 DSV4 B300 AgentX MTP 扫描#2536
cquil11 merged 6 commits into
mainfrom
agentx/dsv4-b300-mtp-upd

Conversation

@ivanium

@ivanium ivanium commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update the B300 DeepSeek-V4-Pro AgentX MTP image to vllm/vllm-openai:nightly-dev-x86_64-cu13.0.1-426e59f.
  • Keep TP8 [1, 4] and TP4 c8 GPU-resident; run TP4 c16 with vLLM SimpleCPU offload.
  • Run DEP4 SimpleCPU at [48, 64] and DEP8 SimpleCPU at [128, 256, 384, 512, 576].
  • Set --long-prefill-token-threshold 512 for DEP4 and DEP8. Every offloaded rank receives a 356 GB host-DRAM budget.

Validation

  • python -m pytest utils/matrix_logic/ -v — 224 passed.
  • Generated and validated all 11 dsv4-fp4-b300-vllm-agentic-mtp rows.
  • bash -n and git diff --check passed.

中文说明

  • 将 B300 DeepSeek-V4-Pro AgentX MTP 镜像更新为 vllm/vllm-openai:nightly-dev-x86_64-cu13.0.1-426e59f
  • TP8 [1, 4] 与 TP4 c8 保持 GPU 常驻;TP4 c16 使用 vLLM SimpleCPU 卸载。
  • DEP4 SimpleCPU 使用 [48, 64],DEP8 SimpleCPU 使用 [128, 256, 384, 512, 576]
  • DEP4 与 DEP8 统一设置 --long-prefill-token-threshold 512;每个卸载 rank 分配 356 GB 主机 DRAM。

验证

  • python -m pytest utils/matrix_logic/ -v:224 项测试通过。
  • 已生成并验证 dsv4-fp4-b300-vllm-agentic-mtp 的全部 11 条配置。
  • bash -ngit diff --check 均通过。

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

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 As a PR reviewer and CODEOWNER, I have reviewed this and have.

For PR verification, add the full-sweep-fail-fast label (strongly recommended) to this PR — the benchmark sweep only runs on labeled PRs. Use full-sweep-enabled only if you need matrix jobs to keep running past a failure.

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 模板,包括保留英文语句 As a PR reviewer and CODEOWNER, I have reviewed this and have

如需进行 PR 验证,请为此 PR 添加 full-sweep-fail-fast 标签(强烈推荐)— 基准测试 sweep 仅在带有标签的 PR 上运行。仅当需要矩阵任务在失败后继续运行时才使用 full-sweep-enabled

PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档

ivanium added a commit that referenced this pull request Aug 8, 2026
Replace the temporary changelog placeholder with draft PR #2536.

中文:将变更日志中的临时占位符替换为草稿 PR #2536 的链接。
@ivanium
ivanium force-pushed the agentx/dsv4-b300-mtp-upd branch from d1b6a57 to dff6feb Compare August 8, 2026 21:12
@ivanium
ivanium marked this pull request as ready for review August 8, 2026 21:14
@ivanium
ivanium requested a review from a team August 8, 2026 21:14
@ivanium ivanium added full-sweep-fail-fast agentx AgentX benchmarks, recipes, and infrastructure labels Aug 8, 2026
@claude

claude Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @ivanium's task in 1m 34s —— View job


PR Review

  • Gather context (diff vs origin/main, changed files)
  • Review benchmark script changes (dsv4_fp4_b300_vllm_mtp.sh)
  • Validate configs/nvidia-master.yaml changes (image, prefixes, sweep grid)
  • Validate perf-changelog.yaml entry placement
  • Post review feedback

LGTM - no blocking issues found

Checks performed:

  • perf-changelog.yaml entry is correctly appended at the end of the file and links this PR.
  • New image vllm/vllm-openai:nightly-dev-x86_64-cu13.0.1-426e59f is a valid public Docker Hub reference (tag existence not verified remotely from this sandbox — please confirm the nightly tag is still published, since nightly-dev tags can be pruned).
  • Sweep grid: 2 + 3 + 3 + 6 = 14 rows, matching the stated validation; all DEP conc values satisfy the 2*CONC % TP == 0 guard in the script (DEP4: 40/48/64, DEP8: 64–384).
  • Script keeps --enable-expert-parallel conditional on EP_SIZE, the vllm serve command remains multi-line, and the moved --long-prefill-token-threshold 512 (shared by DEP4/DEP8) is consistent with the updated comments; DEP8 retains its distinct --max-num-batched-tokens 16384 and 0.92 GPU-mem-util.
  • The DEP8 config comment fix ("SimpleCPU" → "GPU-resident") now matches its kv-offloading: none setting.

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM — this exact change is already merged to main as commit 804ab8b ("perf(agentx): retune DSV4 B300 MTP sweep"), and the follow-up changelog-link commit dff6feb references this PR.

Extended reasoning...

Overview

This PR retunes the DSV4-Pro B300 AgentX MTP benchmark recipe: bumps the vLLM nightly image tag, reshapes the TP4/TP8/DEP4/DEP8 concurrency grids (dropping the TP4-SimpleCPU arm, adding higher-concurrency DEP8 points), raises DEP4 host-DRAM utilization from 0.80 to 0.95, and unifies --long-prefill-token-threshold 512 across both DP-attention arms. Only configs/nvidia-master.yaml, the corresponding benchmark shell script, and perf-changelog.yaml are touched.

Security risks

None. This is benchmark/perf-sweep configuration for an internal measurement harness — no auth, credential, or user-facing logic is touched.

Level of scrutiny

Low-to-moderate is appropriate: this is a routine performance-tuning PR of the kind that lands regularly in this repo. I verified the DEP4 DRAM-budget comment update is internally consistent with agentic_dram_offload_gb in utils/matrix_logic/generate_sweep_configs.py (utilization scales the total budget linearly, and 1424 GB / 356 GB-per-rank at 0.95 tracks the prior 280 GiB-per-rank figure at 0.80 within expected unit-conversion rounding), and that the new --long-prefill-token-threshold 512 was correctly hoisted out of the DEP8-only branch into the shared DP-attention branch so it now applies to both DEP4 and DEP8 as the description states. All new conc-list values for DP-attention arms satisfy the script's 2*CONC % TP == 0 guard.

Other factors

Notably, git log shows this exact diff already merged to main as commit 804ab8b ("perf(agentx): retune DSV4 B300 MTP sweep"), with a subsequent commit dff6feb linking this PR number into the changelog — indicating the sweep already ran and passed. There is nothing outstanding to flag.

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@xinli-sw

xinli-sw commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

/stage-results 31278845561

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@xinli-sw staged run 31278845561: https://inferencemax-app-git-staging-semianalysisai.vercel.app/inference?i_dates=2026-08-08~r31278845561

This run remains available across future /stage-results requests. Staging the same run ID again updates its staged data. Staging workflow

@xinli-sw 已将运行 31278845561 发布到预发布环境:https://inferencemax-app-git-staging-semianalysisai.vercel.app/inference?i_dates=2026-08-08~r31278845561

后续的 /stage-results 请求不会移除此运行;再次发布相同的运行 ID 会更新其预发布数据。预发布工作流

ivanium added a commit that referenced this pull request Aug 9, 2026
Replace the temporary changelog placeholder with draft PR #2536.

中文:将变更日志中的临时占位符替换为草稿 PR #2536 的链接。
@ivanium
ivanium force-pushed the agentx/dsv4-b300-mtp-upd branch from 7299790 to affc6c3 Compare August 9, 2026 01:07
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

1 similar comment
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@ivanium
ivanium force-pushed the agentx/dsv4-b300-mtp-upd branch from 285ea3e to affc6c3 Compare August 10, 2026 17:33
Update the vLLM image and concurrency grid, raise the DEP4 host-memory budget, and set long-prefill-token-threshold to 512 for DEP4 and DEP8.

中文:更新 vLLM 镜像与并发配置,提高 DEP4 主机内存预算,并为 DEP4 和 DEP8 将 long-prefill-token-threshold 设为 512。
Replace the temporary changelog placeholder with draft PR #2536.

中文:将变更日志中的临时占位符替换为草稿 PR #2536 的链接。
Move every DEP8 point to vLLM SimpleCPU offload, widen the concurrency grid through c512, and drop DEP4 c40.

中文:将全部 DEP8 配置切换到 vLLM SimpleCPU 卸载,将并发扫描扩展至 c512,并移除 DEP4 c40。
Move TP4 c16 to SimpleCPU offload, skip TP4 c12 and DEP8 c64, and extend the DEP8 sweep through c576.

中文:将 TP4 c16 切换到 SimpleCPU 卸载,移除 TP4 c12 与 DEP8 c64,并将 DEP8 扫描扩展至 c576。
@ivanium
ivanium force-pushed the agentx/dsv4-b300-mtp-upd branch from affc6c3 to 63ea0c9 Compare August 10, 2026 17:47
@github-actions

Copy link
Copy Markdown
Contributor

1 similar comment
@github-actions

Copy link
Copy Markdown
Contributor

@ivanium

ivanium commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator Author

/stage-results 31415828111

@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

@ivanium staged run 31415828111: https://inferencemax-app-git-staging-semianalysisai.vercel.app/inference?i_dates=2026-08-10~r31415828111

This run remains available across future /stage-results requests. Staging the same run ID again updates its staged data. Staging workflow

@ivanium 已将运行 31415828111 发布到预发布环境:https://inferencemax-app-git-staging-semianalysisai.vercel.app/inference?i_dates=2026-08-10~r31415828111

后续的 /stage-results 请求不会移除此运行;再次发布相同的运行 ID 会更新其预发布数据。预发布工作流

@cquil11

cquil11 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

/reuse-sweep-run 31415828111

@cquil11
cquil11 merged commit c62f6e3 into main Aug 11, 2026
27 checks passed
@cquil11
cquil11 deleted the agentx/dsv4-b300-mtp-upd branch August 11, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agentx AgentX benchmarks, recipes, and infrastructure full-sweep-fail-fast

Projects

Development

Successfully merging this pull request may close these issues.

3 participants