Skip to content

[NVIDIA][AgentX] Add GLM-5.2 NVFP4 B300 Dynamo+SGLang aggregated TP4 AgentX key / 新增 GLM-5.2 NVFP4 B300 Dynamo+SGLang 聚合 TP4 AgentX 配置 - #3953

Open
hshrivastava-droid wants to merge 2 commits into
mainfrom
glm52-fp4-b300-dynamo-sglang-agentx-tp4
Open

hshrivastava-droid wants to merge 2 commits into
mainfrom
glm52-fp4-b300-dynamo-sglang-agentx-tp4

Conversation

@hshrivastava-droid

Copy link
Copy Markdown
Collaborator

Summary

Adds config glm5.2-fp4-b300-dynamo-sglang-agentic-agg-tp4 (GLM-5.2 NVFP4, B300, Dynamo + SGLang, aggregated TP4, AgentX).

Image: lmsysorg/sglang:nightly-dev-cu13-20260901-07c8f729

Type of change

  • New benchmark config
  • Config update / bug fix
  • Infrastructure / tooling

Checklist

  • Config added to nvidia-master.yaml
  • perf-changelog.yaml entry added
  • Matrix generation passes for the new key
中文

新增配置 glm5.2-fp4-b300-dynamo-sglang-agentic-agg-tp4(GLM-5.2 NVFP4,B300,Dynamo + SGLang,聚合 TP4,AgentX)。

AI model disclosure

AI-assisted. The exact model identifier could not be verified from the runtime.

🤖 Generated with Claude Code

…AgentX key / 新增 GLM-5.2 NVFP4 B300 Dynamo+SGLang 聚合 TP4 AgentX 配置

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@hshrivastava-droid
hshrivastava-droid requested a review from a team October 13, 2026 06:25
@hshrivastava-droid hshrivastava-droid added the full-sweep-enabled Full sweep with canary gate; matrix jobs run to completion despite failures label Oct 13, 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.

I reviewed this PR and didn't find any bugs. Because it adds a new multi-node SLURM benchmark recipe plus matching master-config and changelog entries across files owned by the NVIDIA srt-slurm-recipes CODEOWNERS, a human look (from one of the listed owners) would still be worthwhile.

What was reviewed:

  • The new agg-tp4-variants.yaml recipe (base + 7 concurrency overrides) against the matching nvidia-master.yaml search-space entries for config-key/override wiring consistency.
  • Container image consistency between the recipe's identity.container.image and the master config's image: field — both pin lmsysorg/sglang:nightly-dev-cu13-20260901-07c8f729.
  • SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE: '0' against CONTRIBUTING.md's draft-precision rule and sibling GLM-5.2 FP4 recipes — all set it to '0', consistent, not an enable.
  • health_check timing (1440 attempts × 10s = 4h) against slurm.time_limit: 4:00:00 and sibling GLM-5.2 recipes — matches the established pattern exactly.
  • perf-changelog.yaml entry — single new append-only block, correct config-keys, matches the one-PR-one-block convention.
Extended reasoning...

The diff is a purely additive three-file change: a new multi-node srt-slurm recipe (GLM-5.2 NVFP4 AgentX on B300, Dynamo+SGLang aggregated TP4, 7 concurrency variants), a matching nvidia-master.yaml search-space entry, and one perf-changelog.yaml append. It touches no auth, crypto, or data-exposure surface — it's infrastructure/benchmark configuration. The deciding factor for deferring rather than approving is that both changed paths (inferencex-e2e/configs/nvidia-master.yaml and the b300-* srt-slurm-recipes tree) are covered by non-core CODEOWNERS entries in .github/CODEOWNERS, and the visible PR timeline shows no sign-off from those owners yet.

@github-actions

Copy link
Copy Markdown
Contributor

@nvpohanh

Copy link
Copy Markdown
Collaborator

On my radar. I will update the cookbook tomorrow and run the sign-off process

@nvpohanh

Copy link
Copy Markdown
Collaborator

[by Claude Code] The SGLang cookbook integration of this PR's AgentX config (glm5.2-fp4-b300-dynamo-sglang-agentic-agg-tp4) will happen in sgl-project/sglang#44065, in the Agentic Long-Context Deployment section of the GLM-5.2 cookbook page.

This PR has no authorized /use <run_id> comment yet, so the cookbook shows the recipe at this PR's head (caa2199eb) as "in progress". It switches to the measured run at the next cookbook sync after a run is pinned or this PR merges.

…-sglang-agentx-tp4

Resolve the conflicts in configs/nvidia-master.yaml and perf-changelog.yaml.

Main now selects srt-slurm recipes with srt-recipe-dir and srt-recipe and
composes AgentX recipes from fragments (#3805, #3818), so this entry and its
bundle move to that format:
- The master entry sets srt-recipe-dir, and each row selects its override with
  srt-recipe instead of a CONFIG_FILE setting.
- The bundle drops the keys the binder and launcher now write: the model
  block, the identity repo and image, the client command,
  benchmark.concurrencies, RESULT_DIR, IS_MULTINODE, ENABLE_AGENTX_POWER,
  AIPERF_DATASET_MMAP_CACHE_DIR and HF_HUB_CACHE. It also drops the unread
  INFMAX_CONTAINER_WORKSPACE and PORT.

No serving setting changes: engine args and env, frontend, Dynamo, services
and Slurm options bind exactly as the PR's sweep submitted them. The
perf-changelog entry stays appended at the end of the file.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nvpohanh nvpohanh removed the full-sweep-enabled Full sweep with canary gate; matrix jobs run to completion despite failures label Oct 14, 2026

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

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants