Conversation
Point the dependency remote at SemiAnalysisAI while retaining the existing runtime pin. Document submodule synchronization for existing checkouts. 中文:使用 srt-slurm 分支仓库启动 v2 重构。将依赖远程地址指向 SemiAnalysisAI,保留原有运行时版本,并补充现有检出目录的子模块同步说明。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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-slurmatSemiAnalysisAI/srt-slurmand documents remote synchronization for existing checkouts. It retains the exact runtime pin984180e5b8755aef85e9995048b5a16cb5336bce(upstream v2.2.1), so this first commit changes the dependency source without changing benchmark execution.Work remaining
mainis two commits behind upstream5df10653f70b8076b64c8cd261389d27cf4dbeb0; changing the remote alone does not perform that sync.additional-settings.CONFIG_FILE.Validation
git diff --checkpassed.utils/srt-slurmfrom the SemiAnalysisAI fork and verified checkout of the unchanged pinned commit.Related Issue
#2382
Type of Change
Checklist
perf-changelog.yamland have not edited historical entries. Not applicable to the starter commit; required for subsequent runtime/recipe changes./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 提交,因此本次仅改变依赖来源,不改变基准执行行为。后续工作
main比上方列出的上游提交落后两个提交;修改远程地址本身不会完成同步。additional-settings.CONFIG_FILE。验证与状态
git diff --check通过;已从 SemiAnalysisAI 分支仓库初始化子模块,并确认检出原有固定提交。GPU 验收留待迁移阶段完成。本次初始提交不升级运行时或修改配方,因此暂不需要性能变更日志;后续涉及运行时或配方的变更必须追加日志。关联 issue 为 #2382;变更类型为配置与文档更新。本 PR 保持草稿,完整 sweep、eval 和复用合并条件尚未满足。