Skip to content

[AgentX] MiniMax-M3.1-Flash B200/GB200/B300 SGLang: add conc 1 low-latency configs / [AgentX] MiniMax-M3.1-Flash B200/GB200/B300 SGLang:新增 conc 1 低延迟配置 - #4509

Open
ispobock wants to merge 3 commits into
mainfrom
minimaxm31flash-sglang-agentx-low-latency
Open

ispobock wants to merge 3 commits into
mainfrom
minimaxm31flash-sglang-agentx-low-latency

Conversation

@ispobock

@ispobock ispobock commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

Add a Low-Latency point at conc 1 for the B200, GB200 and B300 SGLang AgentX configs, following the GB300 recipe from #4456: TP4 with the MSA-V1 MegaKernel serving batch-size-1 DSpark steps. Each SKU gets its own -low-latency config key so the sweep covers only the new conc 1 point.

中文

为 B200、GB200、B300 的 SGLang AgentX 配置新增 conc 1 低延迟点,沿用 #4456 中 GB300 的设置:TP4,由 MSA-V1 MegaKernel 处理 batch size 1 的 DSpark 步骤。每个 SKU 使用单独的 -low-latency config key,sweep 只覆盖新增的 conc 1 点。

AI model disclosure

Prepared with Claude Code using claude-opus-5-5. Reviewed by the human author before submission.

…ang AgentX

Adds the MSA-V1 MonoKernel TP4 point at conc 1, following the GB300 recipe.

为 MiniMax-M3.1-Flash 的 B200、GB200、B300 SGLang AgentX 新增 conc 1 低延迟配置,沿用 GB300 的 MSA-V1 MonoKernel TP4 设置。
@ispobock
ispobock requested a review from a team October 25, 2026 12:45
@ispobock ispobock added the full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended) label Oct 25, 2026
@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 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

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

Nothing blocking. The comments below are optional suggestions. There is no need to push a fix for them before merging.

Comment on lines +24 to +25
# Follows the GB300 Low-Latency cell: the MSA-V1 MonoKernel serves batch-size-1
# DSpark steps; larger batches take the standard path.

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.

🟡 nit (optional): the new recipes add a comment saying where the config came from, which AGENTS.md's "No history" rule bans. sweep:# Follows the GB300 Low-Latency cell The comment above args: says what the recipe was copied from. The rule says derivation belongs in git and perf-changelog. Fix: drop "Follows the GB300 Low-Latency cell:" from each of the three new recipes. Keep only the technical part, for example # MSA-V1 MonoKernel serves batch-size-1 DSpark steps; larger batches take the standard path.

Why this was flagged

The three new recipe files b200-fp4-mtp/agentic-low-latency.yaml:24, b300-fp4-mtp/agentic-low-latency.yaml:24 and gb200-fp4-mtp/agentic-low-latency.yaml:24 each add # Follows the GB300 Low-Latency cell: the MSA-V1 MonoKernel serves batch-size-1. AGENTS.md "Code comments (mandatory)" covers recipes and says: "Do not write comments that describe how the code evolved, where it was derived from, what it replaced". The opening clause describes where the recipe was derived from, not why a value was chosen. Behaviour does not change. The cost is a convention breach that leaves maintainers a stale pointer once the GB300 recipe changes.

Verification: Each new recipe (b200-fp4-mtp, b300-fp4-mtp and gb200-fp4-mtp agentic-low-latency.yaml) adds the same two lines at 24-25, just above args:, starting "# Follows the GB300 Low-Latency cell". Line 65 of AGENTS.md says: "No history. Do not write comments that describe... where it was derived from". That clause is exactly what that rule bans. No runtime behavior is affected.

@github-actions

Copy link
Copy Markdown
Contributor

Keeps only the technical note, per the no-history comment rule.

只保留技术说明,符合不写来源历史的注释规则。
@ispobock

Copy link
Copy Markdown
Collaborator Author

/use 40194550246

@github-actions

github-actions Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

This branch has not been deployed

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

Labels

full-sweep-fail-fast Full sweep with canary gate; first failure cancels the rest of that matrix (recommended)

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant