aiaccel icon indicating copy to clipboard operation
aiaccel copied to clipboard

Which module should cast the parameter types?

Open styamamoto opened this issue 1 year ago • 1 comments

パラメータの型キャストが何度も起こっているようです.

特に気になった箇所は 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

styamamoto avatar Mar 08 '23 04:03 styamamoto