Skip to content

v2 rework - #3351

Closed
adibarra wants to merge 1 commit into
mainfrom
codex/v2-rework
Closed

adibarra wants to merge 1 commit into
mainfrom
codex/v2-rework

Conversation

@adibarra

Copy link
Copy Markdown
Collaborator

Description

Start the full-YAML rework with SRT-Slurm as the execution component. This draft is the shared integration PR for the work tracked in #2382.

The initial change points utils/srt-slurm at SemiAnalysisAI/srt-slurm and documents remote synchronization for existing checkouts. It retains the exact runtime pin 984180e5b8755aef85e9995048b5a16cb5336bce (upstream v2.2.1), so this first commit changes the dependency source without changing benchmark execution.

Work remaining

Validation

  • git diff --check passed.
  • Initialized utils/srt-slurm from the SemiAnalysisAI fork and verified checkout of the unchanged pinned commit.
  • GPU qualification remains part of the migration work. No runtime upgrade or recipe change is included in this starter commit, so no performance-changelog entry is needed yet.

Related Issue

#2382

Type of Change

  • Bug fix
  • New feature
  • Configuration change
  • Documentation update
  • Other (please describe)

Checklist

  • 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. Not applicable to the starter commit; required for subsequent runtime/recipe changes.
  • Before merging via reuse, an authorized maintainer has commented /use <run_id> (or the legacy /reuse-sweep-run) after a successful full sweep with evals. Pending; this is a draft.
中文

说明

以 SRT-Slurm 作为执行组件,启动全 YAML 重构。本草稿作为 #2382 所跟踪工作的共享集成 PR。

初始变更将 utils/srt-slurm 的来源改为 SemiAnalysisAI/srt-slurm,并更新现有检出目录同步远程地址的说明。运行时仍固定在上方列出的上游 v2.2.1 提交,因此本次仅改变依赖来源,不改变基准执行行为。

后续工作

验证与状态

git diff --check 通过;已从 SemiAnalysisAI 分支仓库初始化子模块,并确认检出原有固定提交。GPU 验收留待迁移阶段完成。本次初始提交不升级运行时或修改配方,因此暂不需要性能变更日志;后续涉及运行时或配方的变更必须追加日志。关联 issue 为 #2382;变更类型为配置与文档更新。本 PR 保持草稿,完整 sweep、eval 和复用合并条件尚未满足。

Point the dependency remote at SemiAnalysisAI while retaining the existing runtime pin. Document submodule synchronization for existing checkouts.

中文:使用 srt-slurm 分支仓库启动 v2 重构。将依赖远程地址指向 SemiAnalysisAI,保留原有运行时版本,并补充现有检出目录的子模块同步说明。
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