現状(PR #54 の時点)では tda_ml/topology.py 内の compute_anisotropic_metric・compute_anisotropic_distance_matrix で軸長パラメータの数値ハック(+1e-6, +1e-8 などの正則化)が入っている。だが、これらのパラメータ調整が本当に必要かどうかは、今後この関数群がどのように使われるか(後続のPR群や利用状況)による。
現時点では:
- テストコードからしか使われていない
- 本格的な用途で呼ばれるのは今後のPRのマージ以降になる
- 数値ハックを残すべき根拠が薄い状態
よって、全ての関連PR(モデル・損失・トレーナ等含む)がマージされた段階で一度「これらの +1e-6/+1e-8 等の正則化パラメータが本当に必要か」を再検証し、必要性が薄ければ削除/設計見直しを検討すること。
- 一時的な対策として入れたものなら追跡必須
- 実運用バグ・分岐条件クリア不能など現象として現れるか注視する
- 必要性判明時は理由・証拠も追記のこと
後続PR #55 #56 #57 #58 が全て入りきった段階で必ず確認タスクとして再検討するためのリマインダーです。
現状(PR #54 の時点)では
tda_ml/topology.py内のcompute_anisotropic_metric・compute_anisotropic_distance_matrixで軸長パラメータの数値ハック(+1e-6,+1e-8などの正則化)が入っている。だが、これらのパラメータ調整が本当に必要かどうかは、今後この関数群がどのように使われるか(後続のPR群や利用状況)による。現時点では:
よって、全ての関連PR(モデル・損失・トレーナ等含む)がマージされた段階で一度「これらの
+1e-6/+1e-8等の正則化パラメータが本当に必要か」を再検証し、必要性が薄ければ削除/設計見直しを検討すること。後続PR #55 #56 #57 #58 が全て入りきった段階で必ず確認タスクとして再検討するためのリマインダーです。