Toma0916 / GlobalWheatDetection

3 stars 0 forks source link

[おわり]: MLflowの名前見やすくする #73

Closed kminoda closed 4 years ago

kminoda commented 4 years ago

68

kminoda commented 4 years ago
スクリーンショット 2020-05-22 13 07 25

BASE_CONFIG.jsonに、デフォルトの値と表示名をリスト形式でそれぞれ書いていく感じにしてみた alphabet順になってしまうのうざいがそれいがいはよさげ? augmentationは使っているのしか書いてない

kminoda commented 4 years ago

https://github.com/Toma0916/GlobalWheatDetection/blob/mlflow_good_design/sample_json/BASE_CONFIG.json これ

kminoda commented 4 years ago
スクリーンショット 2020-05-22 13 59 35

prefixつけつけた いつでもまーじしてよかです

Toma0916 commented 4 years ago

スクリーンショット 2020-05-22 15 07 01

prefixずれますよお兄さん!!!

複数選びうるものは同じprefixにしないとですね。 あとprefix英語だと3文字くらいの時ちょっとややこしい感感ある

kminoda commented 4 years ago

あー 俺そのモードじゃなくて、縦型にパラメータを羅列するモードで考えてたから大丈夫だったけど、そっちのモードにするとやばいのか

kminoda commented 4 years ago
スクリーンショット 2020-05-22 15 13 01

これね

Toma0916 commented 4 years ago

ちなみに僕は割と列ごとにソートしたいので縦でも綺麗に出したいですわね

Toma0916 commented 4 years ago

というかprefix違うとみのだの想定モードでもソートできないと思った

kminoda commented 4 years ago

あ、確かに 誤解してました

kminoda commented 4 years ago

番号つけるのだるいな

kminoda commented 4 years ago

そもそも番号を自動で割り振る感じにすると、今後機能を増やした時にその位置次第では番号がずれて行って、過去の実験結果と対応させることが難しくなるので prefixなしでパラメータ名を出力するのが丸いか?(そのかわり順番はバラバラになってしまうが)

Toma0916 commented 4 years ago

BASE_CONFIG.jsonの途中のところでprefix持てば良さげ

kminoda commented 4 years ago

いやBASECONFIGに全部書いておいて、新しい機能は必ず下の方に付け足していくようにすれば対応は取れる、、、? ちなみにprefixがアルファベットなのは、数字にするとaugmentationとか10を超えてしまいわけわからないことになる(prefix間にseparaterとか入れるとそれはそれで文字が伸びてしまい意味がない)

Toma0916 commented 4 years ago
"train": {
        "prefix": "tr",
        "batch_size": [8, "batch_size"],
        "epochs": [10, "epochs"],
        "random_annotation": {
            "drop": {
                "p": [0.0, "rand_annot_drop_p"]
            },
            "vibration": {
                "max_ratio": [0.0, "rand_annot_vibr_maxratio"]
            }
        },
"augment": {
            "prefix": "ag",
            "horizontal_flip": {
                   p=1
             }
            "test_time_augment": [false, "tta"]
        }

だったら tr_ag_horizontal_flip' tr_ag_tta`

みたいなのどうですか?

kminoda commented 4 years ago

あー そうします

Toma0916 commented 4 years ago

prefixの文字は任せるわ

ってかparam保存する時にorder決めれないのかなって思ったけど見つからんわね

kminoda commented 4 years ago

これ横スクロールってどうやるの?

kminoda commented 4 years ago

あ、できた すごい

Toma0916 commented 4 years ago

あとで見ます

Toma0916 commented 4 years ago

まーじする