Skip to content

Update Qwen3.5 GB300 Dynamo-TRT AgentX recipes / 更新 Qwen3.5 GB300 Dynamo-TRT AgentX 配方 - #4452

Open
RohitNagraj wants to merge 5 commits into
mainfrom
codex/qwen35-gb300-trt-agentx-recipes
Open

RohitNagraj wants to merge 5 commits into
mainfrom
codex/qwen35-gb300-trt-agentx-recipes

Conversation

@RohitNagraj

Copy link
Copy Markdown
Collaborator

Update qwen3.5-fp4-gb300-dynamo-trt-agentic-disagg to ten recipe points using TensorRT-LLM 1.3.0a0.dev202610210406 and Dynamo 1.6.0.dev20261010. The recipes select Qwen3.5 context/generation profiles, explicit NUMA affinity, MTP decode with draft length 6, and native host KV offload.

Upgrade the shared srt-slurm pin from v2.59.1 to v2.60.1 for numa_memory_bind: preferred and post-eval routing to the resolved public frontend. This changes the runtime used by all srt-slurm jobs, including eval placement for other recipes with a decode-node frontend. Both worker roles use iteration-statistics intervals of 20.

AI model disclosure
  • Claude Opus 5.5 (claude-opus-5-5): recipe and configuration authoring.
  • GPT-6 in Codex: configuration editing, local validation, and PR preparation. The exact runtime model/version identifier could not be verified.
  • No delegated agents were used.
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: 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.
中文

将 qwen3.5-fp4-gb300-dynamo-trt-agentic-disagg 更新为十个配方配置点,使用 TensorRT-LLM 1.3.0a0.dev202610210406 和 Dynamo 1.6.0.dev20261010。配方选择 Qwen3.5 context/generation 配置档,显式设置 NUMA 亲和性,decode 使用 draft 长度为 6 的 MTP,并启用原生主机 KV 卸载。

将共享 srt-slurm 从 v2.59.1 升级至 v2.60.1,以支持 numa_memory_bind: preferred,并让 post-eval 连接到已解析的公共 frontend。此升级影响所有 srt-slurm 作业使用的运行时,也会改变其他将 frontend 放在 decode 节点上的配方的评测位置。prefill 和 decode 的迭代统计间隔均设为 20。

改动类型为配置与文档更新。作者检查清单见上方折叠区。

AI 模型披露:Claude Opus 5.5(claude-opus-5-5)用于配方和配置编写;Codex 中的 GPT-6 用于配置编辑、本地校验和 PR 准备,无法核实运行时的精确模型标识及版本。未使用委派代理。

更新 Qwen3.5 GB300 Dynamo-TRT AgentX 配方、TensorRT-LLM 镜像及 srt-slurm 版本。
@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 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

为 Qwen3.5 GB300 AgentX 变更日志条目填写 PR 链接。
@RohitNagraj RohitNagraj added the non-canary-full-sweep-enabled Run the full sweep without the canary gate (full search space, no trim) label Oct 23, 2026
Comment on lines +11064 to +11068
- config-keys:
- qwen3.5-fp4-gb300-dynamo-trt-agentic-disagg
description:
- "Update Qwen3.5 GB300 Dynamo-TRT AgentX recipes, TensorRT-LLM image, and NUMA placement"
pr-link: https://github.com/SemiAnalysisAI/InferenceX/pull/XXX

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.

🔴 Other srt-slurm configs pick up the srt-slurm v2.60.1 runtime with no sweep or changelog record. Only the Qwen3.5 key is covered. This entry lists only qwen3.5-fp4-gb300-dynamo-trt-agentic-disagg, and its description never mentions the pin bump. Fix: describe the upgrade from v2.59.1 to v2.60.1 in this PR's single block. Also list every config key whose run it changes, at least the ones with a decode-node frontend (such as the dsv4 trtllm gb300 AgentX recipe), or use "*" as the v2.43.4 pin-bump entry did.

Why this was flagged

The diff moves inferencex-e2e/utils/srt-slurm from ae3d687 (v2.59.1) to 3ce1c43 (v2.60.1). Every srt-slurm job clones that submodule (RECIPES.md:5). The PR description says this changes the runtime for all srt-slurm jobs, including post-eval routing for other recipes that put the frontend on a decode node. benchmarks/multi_node/srt-slurm-recipes/dsv4/trtllm/gb300-fp4/agentx/disagg-variants.yaml:147 is one such recipe (node: first_decode). infx.matrix.changed only snapshots configs and recipe dirs (revision.py:34-40), so it cannot detect a submodule bump. AGENTS.md:140 says to judge such changes by what they run, and the entry is what triggers result collection. With only the Qwen3.5 key listed (perf-changelog.yaml:11065), no sweep validates the other configs on the new runtime. The published results for those configs also have no record of the runtime change. The earlier v2.39.1 to v2.43.4 bump used config-keys: ["*"] and named the upgrade (perf-changelog.yaml:9344-9347).

Verification: The diff moves the submodule inferencex-e2e/utils/srt-slurm from ae3d6877c46f… to 3ce1c43677921dd4…, which every srt-slurm job clones (RECIPES.md:3). The only new changelog block (perf-changelog.yaml:11064-11068) lists only qwen3.5-fp4-gb300-dynamo-trt-agentic-disagg and never mentions the pin bump, though AGENTS.md:140 requires an entry covering the affected config keys.

将共享 srt-slurm v2.59.1 至 v2.60.1 的升级及评测位置调整写入变更日志,并使用通配符覆盖所有配置。
@github-actions

Copy link
Copy Markdown
Contributor

richardhuo-nv and others added 2 commits October 26, 2026 09:49
Resolve the perf-changelog.yaml append conflict by keeping main's entries
and moving this PR's entry to the end. Picks up #4438, which fixes the
single-node recipe matching that failed check-changelog.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ntX config

A "*" entry plans every config, and the resulting search-space-config
exceeds GitHub's job output size limit, failing setup.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

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

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants