Skip to content

[AgentX] add Muse Spark Open H100 concurrency 1–64 sweep / [AgentX] 新增 Muse Spark Open H100 并发 1–64 扫描 - #4149

Merged
Oseltamivir merged 17 commits into
mainfrom
feat/musespark-h100-agentx
Oct 16, 2026
Merged

Oseltamivir merged 17 commits into
mainfrom
feat/musespark-h100-agentx

Conversation

@edwingao28

@edwingao28 edwingao28 commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Add H100 TP8/EP8 Muse Spark Open NVFP4 with Marlin W4A16. Full sweep: C1/2/4/8/16/24/32/48/64 plus C64 natural GSM8K. Cap decode graphs at 8; use eager DFlash sampling above graph capacity.

Testing: Prior C48/C64 startup hit NCCL OOM. Replacement sweep at 266c72c1f passed setup; GPU qualification remains pending.

Blocker: Full results, cookbook, verified reuse and human review.

中文

新增 H100 TP8/EP8 Muse Spark Open NVFP4,使用 Marlin W4A16。完整扫描覆盖 C1/2/4/8/16/24/32/48/64,以及 C64 自然验证 GSM8K。解码图批量上限为 8;超过图容量时使用 eager DFlash 采样。

**测试:**此前 C48/C64 启动时出现 NCCL OOM。266c72c1f 的替代扫描已通过 setup,GPU 验收仍待完成,链接见上方。

**阻塞:**完整结果、上游 cookbook、验证后的结果复用及人工审查。

AI model disclosure
  • Model/version: Exact model identifiers were not exposed and could not be verified.
  • Role: AI-assisted implementation and review, including delegated agents.
PR checklist and change type

Type of Change

  • Bug fix
  • New feature
  • Configuration change
  • Documentation update
  • Other (please describe)

Checklist

  • I have completed the AI model disclosure and kept it current
  • I have tested my changes locally
  • I have updated documentation if necessary
  • If python -m infx.matrix.changed --base origin/main lists affected config keys, I have appended one new entry covering them to the physical end of inferencex-e2e/perf-changelog.yaml; if it lists none, I have added none. I have not edited historical entries
  • If this PR appends an inferencex-e2e/perf-changelog.yaml entry, it carries exactly one primary sweep label (a maintainer applies it on fork PRs): full-sweep-fail-fast (recommended), full-sweep-enabled, or non-canary-full-sweep-enabled. Optional modifiers all-evals, evals-only, and agentx-fast require a primary label; the last two block reuse while applied.
  • Before merging via reuse, an authorized maintainer (OWNER/MEMBER/COLLABORATOR) has commented /use <run_id> (or the legacy /reuse-sweep-run) on this PR. Do this only once there is a final full sweep that is all green with evals passing, since after this comment the primary sweep label will no longer automatically kick off new sweeps. Remove and re-add the primary sweep label to force a new sweep.

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the contribution!

  • Review: If this PR changes files owned by someone other than a repository admin or @SemiAnalysisAI/core, ask one eligible CODEOWNER to complete the latest PR_REVIEW_CHECKLIST.md before contacting a core maintainer on Slack. Follow the template exactly, including As a PR reviewer and CODEOWNER, I have reviewed this and have, so sign-off verification triggers.
  • PR verification: Sweeps run only when the PR appends an inferencex-e2e/perf-changelog.yaml entry and carries exactly one primary label: full-sweep-fail-fast (strongly recommended; canary plus per-matrix fail-fast), full-sweep-enabled (canary; matrix jobs continue after a failure), or non-canary-full-sweep-enabled (no canary or fail-fast). The modifiers all-evals, evals-only, and agentx-fast require a primary label. On fork PRs, a maintainer applies the label. See sweep labels and reuse.
  • After merging: PR authors must ensure all GitHub Actions jobs pass. Transient failures often pass on rerun; see how to rerun failed jobs.
中文

感谢你的贡献!

  • **审阅:**如果 PR 修改的文件归属于仓库管理员及 @SemiAnalysisAI/core 之外的 CODEOWNER,请先联系一位有资格的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,再通过 Slack 联系核心维护者。必须严格遵循模板,并保留 As a PR reviewer and CODEOWNER, I have reviewed this and have,才能触发签核验证。
  • **PR 验证:**只有当 PR 在 inferencex-e2e/perf-changelog.yaml 末尾追加了条目,并且恰好带有一个主标签时,才会运行扫描:full-sweep-fail-fast(强烈推荐;canary 加逐矩阵 fail-fast)、full-sweep-enabled(有 canary;矩阵任务在失败后继续运行)或 non-canary-full-sweep-enabled(无 canary,也无 fail-fast)。修饰标签 all-evals、evals-only 和 agentx-fast 必须与主标签一起使用。fork PR 的标签由维护者添加。参见扫描标签与复用。
  • **合并后:**PR 作者必须确保所有 GitHub Actions 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

@edwingao28 edwingao28 added the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Oct 16, 2026
@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

中文:同步主分支并保留已测提交和当前配置。
扩展 H100 并发扫描至 1–64,并同步主分支。
同步 B300 主分支集成并保留 H100 完整并发扫描。
同步 GB300 主分支集成并保留 H100 完整并发扫描。
@edwingao28 edwingao28 changed the title [AgentX] add Muse Spark Open H100 at concurrency 1 / [AgentX] 新增 Muse Spark Open H100 并发 1 配置 [AgentX] add Muse Spark Open H100 concurrency 1–64 sweep / [AgentX] 新增 Muse Spark Open H100 并发 1–64 扫描 Oct 16, 2026
edwingao28 and others added 4 commits October 15, 2026 23:35
限制 H100 解码图批量并保留完整并发扫描。
同步主分支并保留 H100 扫描矩阵及解码图修复。
…gentx

# Conflicts:
#	inferencex-e2e/perf-changelog.yaml
@Oseltamivir
Oseltamivir marked this pull request as ready for review October 16, 2026 07:12
@Oseltamivir
Oseltamivir requested a review from a team October 16, 2026 07:12
@Oseltamivir Oseltamivir added non-canary-full-sweep-enabled Run the full sweep without the canary gate (full search space, no trim) and removed full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) labels Oct 16, 2026
@claude

claude Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @Oseltamivir's task in 34s —— View job


LGTM - no blocking issues found.

What I checked
  • The recipe's zip lists (max-running-requests, cuda-graph-max-bs-decode, CONC) each have 9 entries, which matches the master-config conc-list of C1–C64.
  • TP8/EP8 matches the search space, and SGLang gets --expert-parallel-size 8 set explicitly.
  • The image is a public Docker Hub digest, lmsysorg/sglang:dev-muse-spark-open@sha256:….
  • The musespark prefix is already used by the B200, B300, GB200 and GB300 configs.
  • The model was added to h100-dsxe models.entries, which matches runner: cluster:h100-dsxe.
  • The perf-changelog entry is appended at the end of the file, covers the one new config key, and is not append-only.
  • I couldn't run infx.matrix.changed in this environment because the command needed approval. The new key is covered by the appended entry either way.

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

I reviewed this PR and didn't find any bugs. The new recipe and master entry are in CODEOWNER-owned NVIDIA paths, and the description says the C64 graph-allocation fix hasn't passed a sweep yet, so a human look is still worthwhile.

What was reviewed:

  • Recipe against master entry: srt-recipe-dir, srt-recipe, and TP8/EP8 match. The 9-point zip lists (max-running-requests, cuda-graph-max-bs-decode, CONC) line up with conc-list 1..64.
  • Compared the recipe with the sibling B300 Muse Spark recipe. The draft checkpoint and DFlash2 settings are unchanged, with no draft-dtype overrides and no hard-coded acceptance lengths.
  • The runners.yaml models.entries row is in sorted order. The perf-changelog block is appended at the tail as a single English entry for the new key.
Extended reasoning...

The change adds one new H100 SGLang AgentX srt-slurm recipe, a matching nvidia-master.yaml entry with an image pinned by digest, one runners.yaml model-path row, and an appended perf-changelog entry. It touches no security-sensitive surface. It is configuration only, but .github/CODEOWNERS assigns both nvidia-master.yaml and the h100-* recipe tree to specific NVIDIA owners. The PR description also says the C64 graph-memory fix is still waiting on a sweep, so a human should confirm the results before merging.

Oseltamivir and others added 2 commits October 16, 2026 15:47
c48 started with 0.29 GB free per GPU and OOMed in the vision encoder's
all-gather on the first image prefill.
启用 eager draft sampler,保留解码图批量上限 8 和完整并发扫描。
Batches above the batch-8 decode graphs run eager and OOM in fused MoE
with ~1 GB free per GPU (c48, run 38870885984). Drop c24-c64.
@Oseltamivir

Copy link
Copy Markdown
Collaborator

/use 38878624849

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

@Oseltamivir staged run 38878624849.

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

…e Spark Open H100 添加 HiCache 卸载 (#4181)

* feat: add HiCache offload arm to H100 Muse Spark at c2-c8

* chore: link H100 HiCache entry to PR 4181

@Oseltamivir Oseltamivir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@Oseltamivir
Oseltamivir merged commit 68145e7 into main Oct 16, 2026
27 checks passed
@Oseltamivir
Oseltamivir deleted the feat/musespark-h100-agentx branch October 16, 2026 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

non-canary-full-sweep-enabled Run the full sweep without the canary gate (full search space, no trim)

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants