Skip to content

[SRT runtime] repair setup and required recipe power / 修复安装并按配方校验功耗 - #3429

Closed
edwingao28 wants to merge 3 commits into
mainfrom
fix/glm52-powerx-recovery
Closed

edwingao28 wants to merge 3 commits into
mainfrom
fix/glm52-powerx-recovery

Conversation

@edwingao28

Copy link
Copy Markdown
Collaborator

Description

Repair srtctl installation failures and require recipe-selected AgentX PowerX on B200, GB200 and GB300. Reuse #3358's admission contract; retain strict measurement validation and raw audits.

Testing: 154 focused tests, 37 campaign recipe expansions, and real Hatch timeout regression passed; GPU qualification pending.

Blocker: Shared prerequisite for #3400/#3401/#3402; shared planner selects 538 performance + 404 eval cells, beyond this campaign. Qualification and upstream patch integration remain pending; no sweep dispatched.

AI model disclosure

  • Model/version: GPT-6; exact runtime version unavailable.
  • Role: Implementation, delegated investigation/review, and drafting.

Related Issue

None.

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
  • 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 perf-changelog.yaml and have not edited historical entries
  • 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 sweep label will no longer automatically kick off new sweeps. Remove and re-add the label to force one.
中文

修复 srtctl 安装失败,并让 B200、GB200、GB300 根据选定配方启用必需的 AgentX PowerX。复用 #3358 的接入契约,保留严格测量验证和原始审计。

验证: 154 项针对性测试、37 个 campaign 配方解析及真实 Hatch 超时回归通过;GPU 验收待完成。

待办: 本 PR 是 #3400/#3401/#3402 的共享前置修复;共享 planner 选中 538 个性能点和 404 个评估点,超出本 campaign 范围。验收和上游补丁集成尚未完成;未启动 sweep。

AI 模型: GPT-6,无法核实精确运行时版本;负责实现、委派调查与审查、草拟。无关联 issue;类型为缺陷修复、配置及文档更新。

修复 SRT 安装失败,按选定原生配方接入必需功耗,并保留严格测量验证。
@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 only run on labeled PRs. Add full-sweep-fail-fast (strongly recommended); use full-sweep-enabled only when matrix jobs should continue after a failure.
  • 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 上运行。强烈建议添加 full-sweep-fail-fast;仅当需要矩阵任务在失败后继续运行时才使用 full-sweep-enabled。
  • **合并后:**PR 作者必须确保所有 GitHub Actions 任务通过。临时性失败通常可以通过重新运行恢复;参见重新运行失败任务的说明。

将共享 runtime 修复记录关联至 PR #3429。
修复原有 Git 替身未处理全局参数的问题,保留运行时行为及原断言。
@edwingao28

Copy link
Copy Markdown
Collaborator Author

Superseded by #3400, #3401, and #3402. Required fixes are now included in each model × SKU PR. Sweep and PowerX qualification continue there.

@edwingao28 edwingao28 closed this Sep 25, 2026
@Oseltamivir
Oseltamivir deleted the fix/glm52-powerx-recovery branch October 18, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant