feat: 公式マルチシード再現と CI(pytest + repro-smoke) - #58
Conversation
README / REPRODUCIBILITY と experiments/run_backend_multiseed を追加し、 workflow に pytest と軽量な run_backend_multiseed スモークを載せる。 Co-authored-by: Cursor <cursoragent@cursor.com>
|
依存管理まわりで一点、確認・修正した方がよさそうです。 現状、 git submodule update --init --recursiveを実行する前提の記述になっていますが、この PR head では ただし、 torch_topological = { path = "pytorch-topological", editable = true }としているため、再現性の観点では 推奨は次のどちらかです。
この repo の目的が再現性確保なので、私は 1 の submodule 登録の方が自然だと思います。 |
|
Codex review item 1(backend comparison の confounding)については、これは現時点の実験設計上ある程度避けられないものとして、明文化して扱う方針がよいと思います。 現状では、Mahalanobis backend は であり、「distance backend だけを完全に isolate した比較」ではありません。 ただし、これは必ずしも PR blocker ではないと思います。実装上・数学上、ellphi tangency distance に Mahalanobis 側と同型の probability weighting を直ちに入れるのは自然な一行修正ではなく、むしろ比較対象の定義を曖昧にする危険があります。したがって当面は、README / REPRODUCIBILITY / 実験ログの説明で、次を明確に書くのがよいです。
もし将来的に backend-only ablation を追加するなら、Mahalanobis 側でも |
…veats Use ensure script in docs; document Mahalanobis vs ellphi comparison limits. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Issue 4531819747: |
|
Issue 4531912256: |
Keep strict finite checks for MCC/recall, but tolerate non-finite val_loss when parsing metrics.csv so CI repro-smoke does not fail on NaN loss fields. Co-authored-by: Cursor <cursoragent@cursor.com>
…ture/optimized-tda-src Bring in pytorch-topological pin, Copilot/t-uda fixes across config, topology, ellphi gradient, multiseed driver, and CI repro-smoke handling. Keep feature-branch training code (models/losses with ellipse_param_dim). Co-authored-by: Cursor <cursoragent@cursor.com>
|
スカッシュコミット 5b61c98 を main に直接 push して適用済み(squash merge 累積によるコンフリクト回避)。 |
Rebase feature/optimized-tda-src onto uda-lab/TDA-ML main. Preserves fork-only benchmarks, topology geometry fixes, and experiment scripts while adopting upstream PR stack (uda-lab#52-uda-lab#58). Co-authored-by: Cursor <cursoragent@cursor.com>
Summary
experiments/run_backend_multiseed.pyを追加し、読者向けにREPRODUCIBILITY.mdとREADME.md(Known Constraints 含む)を整備する。.github/workflows/ruff.ymlにpytestと、run_backend_multiseed.pyの軽量 repro-smoke(1 epoch・seed 1・mahalanobis のみ)を追加し、README / REPRODUCIBILITY の CI 記述と一致させる。Depends on
mainに対して PR7 分のみの差分に収束する想定。)Test plan
uv run pytestと README の smoke コマンドが通るMade with Cursor