refactor: config・実験スクリプト名を実験設定が読める形に整理 - #7
Open
koki3070 wants to merge 4 commits into
Open
Conversation
elongate/full120 など誤解を招く接頭辞をやめ、 paper_/tune_ + n100_o20_nocls_h1_ellphi_lpca_power に統一。 未使用の baseline と Methods-only barrier YAML は公開面から削除。 Co-authored-by: Cursor <cursoragent@cursor.com>
tune_elongate_* / run_teacher_local_pca_power_* 等を tune_wdist / run_paper_30ep / eval_paper 等へ改名。 出力パス・eval タグ・run_slug も paper/tune 語彙に揃える。 Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
- paper_n100_o20_nocls_* と tune_n100_o20_nocls_* が同一 slug
(h1_ellphi_lpca_power) に潰れ、run ディレクトリ名から config を
特定できなくなっていた。role トークンを残すよう変更。
- 旧 config_id teacher_local_pca_power_seed{N} を paper_s{N} に
再割当てしていたため、正本 paper_seed{N} と run 名前空間が衝突。
ディスク上の実ディレクトリ名 pwr_s{N} に戻して分離を回復。
- prefix 除去も origin/main と同一挙動に戻し、旧 slug を維持。
Co-authored-by: Cursor <cursoragent@cursor.com>
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.
概要
PR #6 マージ後に uda-lab 向け PR(uda-lab#66)で行った命名整理を fork 側 main にも反映する。計算内容は不変(リネーム+参照更新+不要 YAML 削除)+ run-dir slug のバグ修正。
変更内容
configs/ の命名規約導入
paper_n100_o20_nocls_h1_ellphi_lpca_power.yaml(旧elongate_n100_no_cls_full120_teacher_local_pca.yaml): 論文本番 config。点数 n100・外れ値 o20・分類ヘッドなし・H1-only・ellphi backend・local-PCA teacher・power size を名前で明示。tune_n100_o20_nocls_h1_ellphi_lpca_power.yaml(旧elongate_n100_no_cls_tune_local_pca_ellphi_power.yaml): Optuna チューニング共通ベース。aniso_mode=elongate_barrier)は preflight 側の契約定数として残り、BASE_CONFIGで明示宣言した YAML を渡す opt-in 経路のみ(公開 YAML は同梱しない)。configs/README.mdに明記。aniso_mode=elongateは契約デフォルトのためファイル名から除外(命名規約としてconfigs/README.mdに記載)。experiments/ のスクリプト名単純化
tune_elongate_wdist.py/tune_elongate_mcc.pytune_wdist.py/tune_mcc.pyrun_tune_local_pca_power_{wdist,mcc}_parallel.shtune_{wdist,mcc}_parallel.shrun_tune_local_pca_power_objectives.shtune_objectives.shrun_teacher_local_pca_power_30ep.py/_multiseed.shrun_paper_30ep.py/_multiseed.shevaluate_paper_protocol.py/evaluate_paper_baselines.pyeval_paper.py/eval_baselines.pyaggregate_power_30ep_multiseed.pyaggregate_paper_multiseed.pypower_30ep_freshness.pypaper_run_freshness.py出力パス・manifest タグ・
command_entry・Optuna study 名・ドキュメントの参照をすべて更新。本番 run-dir slug もpwr_s{seed}→paper_s{seed}に変更し、freshness / aggregate のグロブと揃えた。run-dir slug のバグ修正(レビュー中に検出)
paper_n100_o20_nocls_*とtune_n100_o20_nocls_*が prefix 除去で同一 slugh1_ellphi_lpca_powerに潰れ、run ディレクトリ名から config を特定できなくなっていた。role トークンを保持するよう修正(paper_h1_ellphi_lpca_pow/tune_h1_ellphi_lpca_powe)。config_idteacher_local_pca_power_seed{N}をpaper_s{N}に再割当てしていたため、正本paper_seed{N}と run 名前空間が衝突していた。ディスク上の実ディレクトリ名と一致するpwr_s{N}に戻して分離を回復。レガシー slug 全体の挙動をmainとバイト一致に復元。検証
uv run pytest: 174 passeduv run ruff check .: All checks passed.shのbash -n構文チェック通過run_paper_30ep.py --tag→eval_paper.pyがlogs/paper_metrics_test_{tag}.jsonを出力 →aggregate_paper_multiseed.py/paper_run_freshness.pyのグロブが一致)outputs/supervised/paper30_{wdist,mcc})と集計スクリプト既定値の一致確認teacher_local_pca_*)とレガシー slug 互換以外ゼロ