aiorhiroki / farmer

farmer is an automated machine learning library.👨‍🌾
Apache License 2.0
12 stars 3 forks source link

Feature/alb augmix #131

Closed sirn524 closed 3 years ago

sirn524 commented 4 years ago

実施したこと

やらないこと

できるようになること(ユーザ目線)

できなくなること(ユーザ目線)

* yamlのaugmentation_statを明記しないと、augmentationが実行されないようになった

動作確認

その他

sirn524 commented 4 years ago

Conflictの解消後、train_paramsのfeatureがマージされたため、train_paramsにaugmix parameter2つを追加 追加要項 ・augmentation_stat --- augmentationの種類が増えるに連れて、yamlでの操作が可能となる ・augmix --- true, falseでmixをするかどうかを決定する。

sirn524 commented 4 years ago

train paramsのパラメータ追加が良かったのかどうかは応相談と思われる。 augmix true-- booleanはいるべきか?まぁ、cosine_decayもbool値あるしOK?

yumion commented 4 years ago

山田さんが起票した、オリジナルのaugmixにkeras ImageDataGeneratorの分岐があったので、そこの追加がしたくて作りました。

これ、ImageDataGeneratorを使っていたときにAugMixを入れただけなので、ImageDataGeneratorは必要ないですよ。 あと、 augment_and_mix は入力画像だけにしかaugmentationをしないので、labelもaugmentationした方を使っていただけると嬉しいです(そっちはalbumentationsで実装もしてますし)

sirn524 commented 4 years ago

まず松崎さんからいただいた指摘部分反映後、label(mask)をaugmixするように組み込みを分解して考えることにします。 ImageDataGeneratorで目新しいAugmentationとかないですかね。。。random erasingとかも組み込みたいので、別featureで追加(ImageDataGenerator以外でも)検討します

yumion commented 4 years ago

ImageDataGeneratorで目新しいAugmentationとかないですかね

ないと思います…

sirn524 commented 4 years ago

指摘いただいたパートはresolveしましたー

sirn524 commented 4 years ago

label,maskの両方がaugment and mixに入るようにしてみた

sirn524 commented 3 years ago

一旦conflictのresolveとclassificationのいらないパートを消す等のreviewいただいたパートは全部修正しました。 Classificationの挙動についてもう少し調べたいので、review指摘頂きつつでも良いですか?