Skip to content

Follow-up: 正則化・安定化のための ad-hoc 微小正定数の要否をPR 54〜58 マージ後に再検証する #59

Description

@t-uda

現状(PR #54 の時点)では tda_ml/topology.py 内の compute_anisotropic_metriccompute_anisotropic_distance_matrix で軸長パラメータの数値ハック(+1e-6, +1e-8 などの正則化)が入っている。だが、これらのパラメータ調整が本当に必要かどうかは、今後この関数群がどのように使われるか(後続のPR群や利用状況)による。

現時点では:

  • テストコードからしか使われていない
  • 本格的な用途で呼ばれるのは今後のPRのマージ以降になる
  • 数値ハックを残すべき根拠が薄い状態

よって、全ての関連PR(モデル・損失・トレーナ等含む)がマージされた段階で一度「これらの +1e-6/+1e-8 等の正則化パラメータが本当に必要か」を再検証し、必要性が薄ければ削除/設計見直しを検討すること。

  • 一時的な対策として入れたものなら追跡必須
  • 実運用バグ・分岐条件クリア不能など現象として現れるか注視する
  • 必要性判明時は理由・証拠も追記のこと

後続PR #55 #56 #57 #58 が全て入りきった段階で必ず確認タスクとして再検討するためのリマインダーです。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions