Skip to content

Recreate Qwen3.5 B300 AgentX recipes for a full sweep / 重建 Qwen3.5 B300 AgentX 配方并运行完整测试 - #3931

Open
functionstackx wants to merge 3 commits into
mainfrom
codex/duplicate-2978-full-sweep
Open

functionstackx wants to merge 3 commits into
mainfrom
codex/duplicate-2978-full-sweep

Conversation

@functionstackx

@functionstackx functionstackx commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Recreate #2978 (source commit 761428402083ded38ae4908e33f7bd6c9b0ef3ee) on the latest fetched main (20c8fba9e) for an independent full sweep. Carry over its five Qwen3.5 NVFP4 B300 Dynamo-SGLang AgentX configurations, pinned lmsysorg/sglang:nightly-dev-cu13-20260901-07c8f729 image, and synthetic-acceptance validation.

The matching shared SquashFS cache on B300 was removed so the image can be imported afresh. Use full-sweep-enabled for the new run, including the normal canary and selected evals.

Difference from #2978: remove SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE from all four recipes, as requested, instead of carrying over the prohibited enabling override. The remaining recipe settings and synthetic-acceptance behavior are unchanged.

AI model disclosure

GPT-6 prepared this duplicate, performed the cache cleanup, and ran local verification. The runtime does not expose a verifiable exact model/version identifier, so the exact version could not be verified. No delegated agents contributed. The source PR does not provide verifiable identities for any models used to prepare its original changes.

PR checklist and change type

Type of Change

  • Bug fix
  • New feature
  • Configuration change
  • Documentation update
  • Other

Checklist

  • I have completed the AI model disclosure and kept it current
  • I have tested my changes locally
  • I have updated documentation if necessary
  • For every change that can affect benchmark performance and every recipe addition or modification, I have appended a new entry to the physical end of inferencex-e2e/perf-changelog.yaml and have not edited historical entries
  • If this PR can affect benchmark performance or adds or modifies a recipe, it carries exactly one primary sweep label (a maintainer applies it on fork PRs): full-sweep-fail-fast, 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. Only do this after a final full sweep is all green with evals passing.
中文

基于最新获取的 main(20c8fba9e)重新创建 #2978(源提交 761428402083ded38ae4908e33f7bd6c9b0ef3ee),以独立运行一次完整 sweep。沿用原有五组 Qwen3.5 NVFP4 B300 Dynamo-SGLang AgentX 配置、固定镜像 lmsysorg/sglang:nightly-dev-cu13-20260901-07c8f729 及合成接受校验逻辑。改动涵盖配方配置、校验逻辑和文档。

已删除 B300 上对应的共享 SquashFS 缓存,以便重新导入镜像。新运行使用 full-sweep-enabled 标签,保留正常的 canary 和默认选定的评测。

与 #2978 的区别: 按要求从全部四个配方中删除 SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE,不再沿用被规范禁止的启用设置。其余配方配置和合成接受行为保持不变。

AI 模型披露:由 GPT-6 完成 PR 复制、缓存清理及本地验证。运行环境未提供可核实的精确模型或版本标识,因此无法确认精确版本。未使用委派代理。源 PR 未提供可核实的原始改动所用模型标识。

Recreate PR #2978 at 7614284 from origin/main, preserving its recipe and synthetic acceptance behavior. Append its performance changelog entry after the current history for a fresh full sweep.

中文:基于最新远端 main 重新创建 PR #2978,保留原有 Qwen3.5 B300 AgentX 配方与合成接受行为,并在当前性能日志末尾追加对应条目,以便重新运行完整扫描。

Co-authored-by: Rohit Pujar Nagraj <rpujarnagraj@nvidia.com>

Co-authored-by: Ankur-singh <ankusingh@nvidia.com>
中文:将复制的 AgentX 完整扫描条目关联到新建的 PR #3931。
@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 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

@functionstackx functionstackx added the full-sweep-enabled Full sweep with canary gate; matrix jobs run to completion despite failures label Oct 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

inferencex-e2e/infx diff: +98 -0

Generally most InferenceX submissions don't need to add changes to the infx package. On the off chance they do, generally the diff is small. Please prompt your agent to clean your PR to keep your diff to only necessary & minimal changes too. We have an increased code quality bar for changes to the infx package.

中文

大多数 InferenceX 提交通常不需要修改 infx 包。即使确实需要,改动通常也很小。请让你的 agent 清理 PR,使改动只包含必要且最小的内容。infx 包的改动有更高的代码质量要求。

Remove SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE from all four B300 AgentX recipes and update this PR performance changelog entry.

中文:从全部四个 B300 AgentX 配方中删除 SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE,并更新本 PR 的性能变更日志。
@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

SGLANG_FLASHINFER_FP4_GEMM_BACKEND: cutlass
SGLANG_MOE_NVFP4_DISPATCH: '1'
SGLANG_CUTEDSL_MOE_NVFP4_DISPATCH: '1'
SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE: '1'

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.

🔴 These new B300 AgentX recipes enable SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE=1, which CONTRIBUTING.md:71-79 explicitly and unconditionally prohibits (precision hacking of the MTP draft head), so merging ships a policy-violating config that a reviewer following the checklist (inferencex-e2e/docs/PR_REVIEW_CHECKLIST.md:44) must reject. The PR description itself admits the conflict is 'unresolved and must be addressed before merge,' but the code still sets the flag in all four new recipes. Fix: unset/disable SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE (or get an explicit, recorded policy exception) in every recipe that sets it before merge.

Why this was flagged

CONTRIBUTING.md:71-79 states enabling SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE is 'explicitly prohibited going forward, including inherited environment, launcher, container, or image defaults' and 'historical runs are not precedent or an exception.' All four new recipe files set this to '1': agg-tp2-b2-mtp5-hicache.yaml:47, agg-tp2-mtp7-hicache-variants.yaml:48, agg-tp2ep2-mtp3-hicache.yaml:47, agg-tp4-mtp6-hicache-variants.yaml:48. On base main these recipes did not exist; merging introduces four configs that violate the current rule, lowering effective MTP draft precision below the shipped checkpoint default while results are reported as compliant AgentX submissions. The PR_REVIEW_CHECKLIST.md:44 'Explicitly verified... not enabled' gate is the safeguard, but it only catches this if a human reviewer manually checks each file; nothing in the diff itself enforces or blocks the setting.

Verification: CONTRIBUTING.md:71-79 explicitly prohibits enabling SGLANG_NVFP4_CKPT_FP8_NEXTN_MOE, and reviewers must verify it is disabled; historical runs are not precedent. All four newly added recipes set the prohibited value '1': agg-tp2-b2-mtp5-hicache.yaml:47, agg-tp2-mtp7-hicache-variants.yaml:48, agg-tp2ep2-mtp3-hicache.yaml:47, agg-tp4-mtp6-hicache-variants.yaml:48. On base main these four files did not exist, so merging introduces four policy-violating configs where none existed.

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-enabled Full sweep with canary gate; matrix jobs run to completion despite failures

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant