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

[H20-DHMO]

該当Issue

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

修正方針

• Toyota Programming Contest 2023 Spring Final はOther Sponsoredにふるい分ける
• トヨタ自動車 実課題プログラミングコンテスト 2023 Spring はMarathonにふるい分ける

スクリーンショット

Other_Sponsored

Marathon

補足

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

[kenkoooo]

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

[hiro1729]

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

[hiro1729]

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

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

[H20-DHMO]

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

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

追加修正方針

• 2023 Summer final はAHC にふるい分ける
• Open Contest はMarathonにふるい分ける
• List上に表示されていた得点を非表示にする
  （2023 Summer finalはratedのためAHCに、オープンコンテストはunratedのためMarathonにしています）

スクリーンショット

AHC

Marathon

List

[H20-DHMO]

@hiro1729 さん

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

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

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

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

[hiro1729]

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

スクリーンショット

追記

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

[H20-DHMO]

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

[hiro1729]

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

[H20-DHMO]

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

現行

• https://github.com/kenkoooo/AtCoderProblems/blob/master/atcoder-problems-frontend/src/utils/ContestClassifier.ts
• https://github.com/kenkoooo/AtCoderProblems/blob/master/atcoder-problems-frontend/src/api/APIClient.ts

修正

• https://github.com/H20-DHMO/AtCoderProblems/blob/%231383-fix-screen_toyota_properly/atcoder-problems-frontend/src/utils/ContestClassifier.ts
• https://github.com/H20-DHMO/AtCoderProblems/blob/%231383-fix-screen_toyota_properly/atcoder-problems-frontend/src/api/APIClient.ts

[hiro1729]

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

[H20-DHMO]

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

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

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

[hiro1729]

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

[kenkoooo]

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