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.

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