Toma0916 / GlobalWheatDetection

3 stars 0 forks source link

working on formatting config with baseline config #66

Closed kminoda closed 4 years ago

kminoda commented 4 years ago

なんかすごい綺麗な関数がかけてしまった

Toma0916 commented 4 years ago

@kminoda WIP付いてるけど完成?

kminoda commented 4 years ago

まだですね

Toma0916 commented 4 years ago

おけ

kminoda commented 4 years ago

コンフリクト解消しときます

kminoda commented 4 years ago

基本はtrain.pyを実行すれば良い。 train.pyでconfigをloadした直後にformatterが働いて、BASE_CONFIG.jsonを基にフォーマットされる(一応簡単にいじるとヤバいファイルなので大文字にしてみたけど別に小文字でも)

元のloadしたjsonを書き換えることはしないけど、代わりにlogdirに出力されるjsonはフォーマットされた奴

kminoda commented 4 years ago

ちなみに、optimizerとかschedulerとかsamplerがそれに当たるけど、「複数ある選択肢の中から1つだけ選び、かつそれぞれの選択肢に特有のパラメータを持つもの」については、{"name":"", "config":{...}}という形をとると仮定してる

Toma0916 commented 4 years ago

あとでみますん

Toma0916 commented 4 years ago

MLFlowとの兼ね合いもみてみたけどたぶん大丈夫かな?

MLFlowは特定の列が抜けてても空欄で補完してくれるからバグることはないけど、例えばaugmentそれぞれでしてない時にp=0 とかいれておきたいなら、BASE_CONFIGに全部書いておいたほうが良いかもって程度

kminoda commented 4 years ago

とりあえずこれでマージして、後で必要そうだったらそうするわ @Toma0916