search

kenkoooo / AtCoderProblems

Extend your AtCoder
https://kenkoooo.com/atcoder/
MIT License
1.39k stars 151 forks source link

トヨタコンを適切にふるい分ける #1393

Closed H20-DHMO closed 9 months ago

H20-DHMO commented 1 year ago

該当Issue

1383 （トヨタコンがOther Contestsに振り分けられている）

修正方針

スクリーンショット

Other_Sponsored

Other_Sponsored

Marathon

Marathon

補足

Issueには記載ありませんでしたがマラソンについてもふるい分けを入れました。 トヨタのマラソンコンテストは今後も開かれる可能性があることから、「toyota-hcから始まるコンテストIDであれば」というふるい分けにしました。 OSSコントリビュートが初めてのため、誤りやお作法としてよくないことをしていればご指摘いただけると幸いです。

kenkoooo commented 1 year ago

ありがとうございます！時間ができ次第すぐに見ますので少々お待ちを…

hiro1729 commented 11 months ago

Summer finalはMarathonが適切なのではないでしょうか。

hiro1729 commented 10 months ago

summer finalがMarathonではなくAGC-Likeに入ってしまっています。

MarathonとOther Sponsoredは最大点数が3000以上、などで分けられると思います。

H20-DHMO commented 10 months ago

追加修正について根本的な解決はしておらず、その場しのぎな修正に近いです。 今後このようなコンテストが増えるならAHC-Likeのクラス分けが必要そうとは感じています。

スクリーンショットを撮ったのでご確認お願いします。

追加修正方針

スクリーンショット

AHC

AHC

Marathon

Marathon

List

List

H20-DHMO commented 10 months ago

@hiro1729 さん

summer finalがMarathonではなくAGC-Likeに入ってしまっています。

今回の修正をローカルで確認した限りは直っているはずです。 勘違いでしたら申し訳ありませんが、公式サイト上で確認しているのでしたら、この修正は反映されていないです。 現在は反映依頼を出しているだけのためです。 kenkoooo さんの確認によってマージされリリースされるかが確定します。 修正の状態を確認する方法は、READMEや以前kenkooooさんの書かれた記事を確認ください。

MarathonとOther Sponsoredは最大点数が3000以上、などで分けられると思います。

申し訳ありません。個人的にはあまり良い修正とは思えませんでした。 まず、ヒューリスティックのコンテストで最大点数が出てしまうこと自体がよろしくない状態でした。 また、将来的に3000点未満のヒューリスティックや3000点以上のアルゴリズムの問題が出ないとは限らないため、点数で判断するのもその場しのぎの修正になります。 現在のクラス分けの処理ではコンテストの情報は確認していますが、コンテスト内の問題までは確認しておらず、問題まで確認すると、今回起きた事例の判断のために処理が複雑になってしまいます。

hiro1729 commented 10 months ago

環境によって異なるようです。 自分はMacを使っていますが Toyota2023 summer finalはAGC-Likeに、 そのOpenはOther Contestsに、 少し前ですがトヨタ自動車 実課題プログラミングコンテスト 2023 SpringもOther Contestsに入っています。

スクリーンショット

スクリーンショット 2023-09-07 17 24 26 スクリーンショット 2023-09-07 17 24 40 スクリーンショット 2023-09-07 17 26 38

追記

サイト上の話と勘違いしてました。すみません。

H20-DHMO commented 10 months ago

そうだったんですね。もしかすると上手く修正出来てないのでしょうか... 念のため確認したいのですが、ローカルの環境にあるソースコードには私の修正が取り込まれていますか。 ご確認お願いします。

hiro1729 commented 10 months ago

すみません、具体的にどこを見ればいいのかわかりません…

H20-DHMO commented 10 months ago

ローカルにある以下の二つ（ContestClassifier.tsだけ確認すれば多分大丈夫です）のファイルがどちらのコードになっていますでしょうか。 修正のコードになっている上でローカルの環境で動作させたAtCoderProblemsの表示が画像のようになっているのであれば、修正はおかしい状態ですね。 もし、現行のコードであるなら、二つのファイルの中身を置き換えて確認していただきたいです。

現行

修正

hiro1729 commented 10 months ago

あ、ローカルでしたか、すみません。間違えてました。 ローカルでどんな感じで実行すればいいのかわかりません…

H20-DHMO commented 10 months ago

ローカルでは確認されていなかったのですね。以前に返答していますが、以下の方法で確認できます。

修正の状態を確認する方法は、READMEや以前kenkooooさんの書かれた記事を確認ください。

基本的にはDocker DesktopをインストールしてREADMEの方法で確認すればよいのですが、難しいと感じたならば気が向いたらで大丈夫です。 上手く説明できたらよいのですが、説明できるほど詳しくなく...

hiro1729 commented 10 months ago

yarnがよく分からないので諦めます。すみませんでした。1つの意見として取り入れていただければと思います。

kenkoooo commented 9 months ago

よくわかんないけどマージします！