Skip to content

[NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6 / 使用 SGLang DSpark6 更新 GB300 DeepSeek-V4-Pro AgentX - #2623

Open
weireweire wants to merge 2 commits into
mainfrom
perf/dsv4-gb300-dspark-agentx
Open

[NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6 / 使用 SGLang DSpark6 更新 GB300 DeepSeek-V4-Pro AgentX#2623
weireweire wants to merge 2 commits into
mainfrom
perf/dsv4-gb300-dspark-agentx

Conversation

@weireweire

@weireweire weireweire commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Refresh the existing GB300 DeepSeek-V4-Pro Dynamo-SGLang AgentX ladder from EAGLE/MTP to DSpark block size 6.
  • Use deepseek-ai/DeepSeek-V4-Pro-DSpark and lmsysorg/sglang:nightly-dev-cu13-20260816-4a6dc267. This nightly already contains the required DSpark performance and MegaMoE changes, so no runtime patch is applied.
  • For throughput runs, inject thinking-on synthetic acceptance length 3.77 from the DeepSeek-V4-Pro-0813 probabilistic DSpark curve. Eval retains SGLANG_RAGGED_VERIFY_MODE=static and real DSpark verification without synthetic acceptance.
  • Retain MegaMoE for DEP workers and use aggregate concurrency 1/4/8 plus disaggregated concurrency 192/400/640/1536, with 8P4D c1536 as the throughput endpoint.
  • Preserve the earlier model curve separately so acceptance values remain model-specific.

Validation

  • srtctl dry-run passed for all six recipes.
  • 235 matrix, schema, recipe, and synthetic-acceptance tests passed locally.
  • 36 changelog tests plus changelog matrix generation passed locally.
  • Ruff, formatting, YAML, and diff checks passed locally.

Reference

中文说明

摘要

  • 将现有 GB300 DeepSeek-V4-Pro Dynamo-SGLang AgentX 配置从 EAGLE/MTP 更新为 block size 6 的 DSpark。
  • 使用 deepseek-ai/DeepSeek-V4-Pro-DSparklmsysorg/sglang:nightly-dev-cu13-20260816-4a6dc267。该 nightly 已包含所需的 DSpark 性能优化和 MegaMoE 改动,因此不再在运行时应用补丁。
  • 吞吐测试根据 DeepSeek-V4-Pro-0813 概率式 DSpark 曲线注入 thinking-on 模拟接受长度 3.77;评估任务保留 SGLANG_RAGGED_VERIFY_MODE=static,使用真实 DSpark 验证且不注入模拟接受长度。
  • DEP worker 继续使用 MegaMoE;聚合配置使用并发 1/4/8,分离式配置使用并发 192/400/640/1536,并以 8P4D c1536 作为最高吞吐点。
  • 旧模型曲线独立保留,避免混用不同模型的接受长度。

验证

  • 全部 6 份 recipe 均通过 srtctl dry-run
  • 本地通过 235 项 matrix、schema、recipe 和模拟接受长度测试。
  • 本地通过 36 项 changelog 测试及 changelog matrix 生成。
  • 本地通过 Ruff、格式、YAML 和差异检查。

参考

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from b8b5b29 to 75c1056 Compare August 16, 2026 03:12
@weireweire
weireweire marked this pull request as ready for review August 16, 2026 03:35
@weireweire
weireweire requested a review from a team August 16, 2026 03:35

@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.

Additional findings (outside current diff — PR may have been updated during review):

  • 🟡 perf-changelog.yaml:5452-5461 — AGENTS.md line 7 requires bilingual PR titles/descriptions (<English title> / <中文标题> plus a Chinese mirror section like ## 中文说明), but PR #2623's title ("[NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6") and body are English-only. Recently merged PRs (#2493, #2486) follow this bilingual convention — please add a Chinese title suffix and a ## 中文说明 section mirroring the summary.

    Extended reasoning...

    What the rule is: AGENTS.md (the single source of truth this repo's CLAUDE.md points to for all guidance) states verbatim in line 7: "PR and GitHub-issue titles & descriptions must be bilingual — include a Simplified Chinese version in addition to English. Title format: <English title> / <中文标题>. In the PR/issue body, follow the English content with its Chinese translation (e.g. a ## 中文说明 section mirroring the summary...)." This applies to every PR in the repository, not an optional style suggestion.

    Where it is violated: PR #2623's title is [NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6 — no / <中文标题> suffix. The description contains only ## Summary, ## Validation, and ## Reference sections in English, with no ## 中文说明 (or equivalent) mirroring the summary.

    Why this is not a false positive: The convention is actively followed elsewhere in this same repo's recent history. git log shows PR #2493 titled "[Klaud Cold] Enact the August 3, 2026 scenario and precision deprecations / 执行 2026 年 8 月 3 日场景与精度下线" and PR #2486 titled "[Klaud Cold] minimaxm3-fp8-gb300-dynamo-vllm-mtp: day-zero GB300 MXFP8 EAGLE3 MTP + FULL_DECODE_ONLY CG / 新增 GB300 MXFP8 EAGLE3 MTP 配方,解码启用 FULL_DECODE_ONLY 图模式" — both bilingual per the mandated format. This confirms the rule is a live, enforced practice, and PR #2623 is an outlier that skipped it.

    Step-by-step proof:

    1. Open AGENTS.md at the repo root, line 7 — read the bilingual title/description requirement.
    2. Open PR #2623's title: [NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6. Search for / followed by CJK characters — none found.
    3. Open PR #2623's description: three English-only ## sections (Summary, Validation, Reference). Search for 中文 or ## 中文说明 — none found.
    4. Compare against git log --oneline -5: #2493 and #2486 both show the <English> / <中文> pattern in their subject lines, confirming this is the repo's actual practice, not a dead-letter rule.
    5. Conclusion: PR #2623 is out of compliance with an explicit, currently-enforced repository convention.

    Impact and fix: This does not affect the functional correctness of the recipe/config changes in the diff — it is purely a process/documentation gap. The fix is mechanical: rename the PR title to append / <中文标题> (e.g., / 刷新 GB300 DeepSeek-V4-Pro AgentX 至 SGLang DSpark6) and add a ## 中文说明 section to the body mirroring the English summary bullets. Since every original source bug for this finding is a [quality]-prefixed convention/documentation issue with no correctness impact, it should be graded nit.

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from c8261b2 to e5935c1 Compare August 16, 2026 04:02
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from e5935c1 to 033ef73 Compare August 16, 2026 04:07
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from 033ef73 to ab65e7f Compare August 16, 2026 04:34
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from ab65e7f to c68dad8 Compare August 16, 2026 05:34
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire weireweire changed the title [NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6 [NV] Refresh GB300 DeepSeek-V4-Pro AgentX with SGLang DSpark6 / 使用 SGLang DSpark6 更新 GB300 DeepSeek-V4-Pro AgentX Aug 16, 2026
@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from c68dad8 to 8a4a0a0 Compare August 16, 2026 13:35
@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

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch 2 times, most recently from a3adb18 to d94a943 Compare August 16, 2026 20:44
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from d94a943 to 71b9873 Compare August 17, 2026 04:06
Root cause:
- The GB300 DeepSeek-V4-Pro AgentX matrix still used the older EAGLE/MTP serving path and an outdated SGLang nightly.
- The previous draft targeted vLLM recipes instead of the existing SGLang topology ladder.

Fix:
- Replace the SGLang aggregate and disaggregated GB300 recipes with DSpark block size 6 on DeepSeek-V4-Pro-DSpark.
- Use nightly-dev-cu13-20260816-4a6dc267, which already includes the required DSpark performance and MegaMoE support, without a runtime patch overlay.
- Force thinking-on AL 3.82 with static ragged verification, retain MegaMoE on DEP workers, and set the 8P4D throughput endpoint to concurrency 1536.

Validation:
- Passed srtctl dry-run for all seven recipes.
- Passed 224 matrix and schema tests.
- Generated the eight targeted matrix jobs at concurrency 1, 4, 8, 32, 192, 400, 640, and 1536.

中文:将 GB300 DeepSeek-V4-Pro AgentX 的 SGLang 配置从 EAGLE/MTP 更新为 DSpark6,使用已包含所需上游修复的 20260816 nightly,不再注入补丁;固定 AL 3.82 与 static ragged verify,并将 8P4D 吞吐端点设为并发 1536。七份 recipe dry-run、224 项矩阵与 schema 测试以及八个目标矩阵任务生成均已通过。
@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from 71b9873 to 1539767 Compare August 17, 2026 04:07
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from 1539767 to 1ab7a23 Compare August 17, 2026 04:22
@github-actions

Copy link
Copy Markdown
Contributor

@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from 1ab7a23 to 346c900 Compare August 17, 2026 05:01
@github-actions

Copy link
Copy Markdown
Contributor

Root cause:
- The sweep still referenced a nonstandard model/tokenizer path, included a low-value concurrency point, and allowed synthetic acceptance controls to leak into evaluation.
- Aggregate TP checkpoint loading and long-lived AgentX replay connections could stall or expire during startup and multi-turn idle gaps.
- Worker cache metrics were not exposed consistently across the recipe ladder.

Fix:
- Use the standard DeepSeek-V4-Pro-0813 model layout and probabilistic DSpark acceptance data, preserve real evaluation behavior, and keep the validated 1152-concurrency endpoint.
- Enable cache reporting on every DSpark worker and checkpoint prefetch on pure-TP aggregate workers.
- Use srt-slurm v1.0.40 and a 900-second nginx/AIPerf connection timeout for the 1152-concurrency recipe.
- Retain the aggregate startup barrier and focused regression coverage.

Validation:
- Passed YAML and shell syntax validation, git diff checks, and the synthetic-acceptance launcher test.
- Passed exact-key NVIDIA matrix expansion.
- Passed srtctl dry-run for all six generated recipes.

中文:统一使用标准 DeepSeek-V4-Pro-0813 模型布局和 0813 DSpark 接收长度;所有 worker 开启 cache report,纯 TP 配置开启 checkpoint prefetch,并用 srt-slurm v1.0.40 与 900 秒连接保活修复高并发多轮回放连接超时。相关矩阵展开、六份 recipe dry-run、YAML/Shell 校验和 launcher 测试均已通过。
@weireweire
weireweire force-pushed the perf/dsv4-gb300-dspark-agentx branch from 346c900 to 86a9eaa Compare August 17, 2026 09:34
@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant