bucket-mode 名を progresskpabs に改名 (旧名は非推奨 alias) - #407
Merged
Merged
Conversation
…lias) 綴りに含まれる 8 は bucket 数ではなく (bucket 数は --num-buckets、既定 9)、 推論エンジン rshogi の canonical 名 progresskpabs とも綴りが割れていたため 名前を実態に揃える。 - BucketMode::parse を単一パース点として追加し、canonical 名 2 種 + 非推奨 alias progress8kpabs を受理 (CLI は alias 使用時に警告を表示) - raw checkpoint reader は格納 bucket_mode を正規化して照合する (旧名で書かれた既存 ckpt の --resume / --eval-only を維持、恒久 alias)。 writer は canonical 名のみ書く。header layout は不変で version bump なし - eval_sfen も同じ alias を受理 - 旧名格納 v8 ckpt の受理 / cross-mode 拒否 / CLI alias 受理のテストを追加 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…nonical 名に固定 - experiment.json の params.bucket_mode に CLI 生文字列を保存していたため、 非推奨 alias で起動した run に旧綴りが記録されていた。パース済み mode の canonical_name() を保存する - BucketMode::parse_cli を追加し、trainer / eval_sfen の --bucket-mode の エラー文と非推奨警告文を単一パース点に集約 (eval_sfen の独自 match を排除) - eval_sfen の引数整合検証を validate_mode_args として分離しテストを追加、 parse_cli の警告文内容 (alias のみ [deprecated]、canonical は警告なし) を テストで固定 - BucketMode の alias コメントから経緯記述を除き入力互換規則のみ残す Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
1 participant
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.
概要
LayerStack の bucket-mode canonical 名を
progress8kpabsからprogresskpabsに改名する。綴りに含まれる「8」は bucket 数を意味しない (bucket 数は
--num-buckets、既定 9) レガシー名で、実態と乖離していた。また推論エンジン rshogi 側の canonical 名は既にprogresskpabsであり、クロスリポで綴りが割れていたのを揃える。変更内容
BucketMode::ProgressKpAbsへ variant rename、canonical_name() = "progresskpabs"BucketMode::parse/parse_cliを単一パース点として追加。CLI (trainer / eval_sfen) のエラー文・非推奨警告文を共通化params.bucket_modeは CLI 生文字列ではなくパース済み mode の canonical 名を記録alias ポリシー
progress8kpabsを恒久 alias として受理 (既存 ckpt の--resume/--eval-onlyを維持)。v7 以前の暗黙値もprogresskpabsとして解釈--bucket-mode[deprecated]警告を表示。削除期限 = 次の大規模学習 campaign 完了後forward 互換の注意
本改名後に書かれた checkpoint (新名格納) は、改名前の binary では resume できない (bucket mode 文字列の不一致エラー)。
検証
bash scripts/local-ci.shフル PASS ×2 回 (d5d826e 時点 / 480e9b2 時点、全 step・GPU テスト含む)cargo fmt --check/cargo clippy --workspace --all-targets -- -D warnings/ CPU テスト (nnue-train 141 / eval-sfen 2 / cli_tests 65 / raw_ckpt 30) green設計ノートは別途管理。
🤖 Generated with Claude Code