aiaccel
aiaccel copied to clipboard
Which module should cast the parameter types?
パラメータの型キャストが何度も起こっているようです.
特に気になった箇所は DB からの読み出し部分で,categorical
パラメータが全て str
にキャストされています.
実行には (今のところ) 差し支えないようですが,TpeOptimizer の実装と整合性がなかったり,コンフィグで数値として choices
に設定した値が .hp ファイルで文字列になっていたりと,見た目があまり良くありません.
- キャスト箇所を AbstractOptimizer のみに限定
- NelderMeadOptimizer でのキャストを internal な int -> categorical.choices のみに限定
- DB 読み出し時のキャストは廃止
とするのが良いと思います.
-
AbstractOptimizer https://github.com/aistairc/aiaccel/blob/05eb2538ba5748f9231245e18054e938f3bee6da/aiaccel/optimizer/abstract_optimizer.py#L261-L278
-
NelderMeadOptimzier https://github.com/aistairc/aiaccel/blob/05eb2538ba5748f9231245e18054e938f3bee6da/aiaccel/optimizer/nelder_mead_optimizer.py#L237-L248
-
Storage https://github.com/aistairc/aiaccel/blob/05eb2538ba5748f9231245e18054e938f3bee6da/aiaccel/storage/storage.py#L146-L153