Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update a pattern to list samples in AtCoder #628

Merged
merged 4 commits into from
Dec 4, 2019
Merged

update a pattern to list samples in AtCoder #628

merged 4 commits into from
Dec 4, 2019

Conversation

kmyk
Copy link
Member

@kmyk kmyk commented Dec 3, 2019

単に .prettyprint の指定を消すと #625 (comment) の問題があります。なので :first-child の制約で置き換えました。

@kawacchu
Copy link
Contributor

kawacchu commented Dec 3, 2019

https://atcoder.jp/contests/s8pc-4/tasks/s8pc_4_dSampleParseErrorが発生しなくなりました。

#620 はHTMLが壊れていることから生じるfalse-positiveへの対処でした。
当該問題のHTMLをみたところ、今回のPRによって、

  • HTMLが壊れていない箇所 -> a old format
  • HTMLが壊れている箇所(<section>タグ抜け) -> the standard format

でパースされ、結果的にユーザに期待されたサンプルを収集しているように見受けられます。
手元で試していないので、確認お願いします。

@kmyk
Copy link
Member Author

kmyk commented Dec 3, 2019

テストの修正をしました。なぜか偶然動いてるだけという状態なのでなんだかこわいですが、これをわざわざエラーにするのも面倒なのでとりあえずこのままにしておきます

@kmyk
Copy link
Member Author

kmyk commented Dec 4, 2019

rebase しました。テスト通ってくれた

Copy link
Contributor

@kawacchu kawacchu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コードは問題ないと思います。
s8pc_4_dのテストは状況に応じて変えていきましょう。
first-childに着目するのは良いアイデアだと思います。ありがとうございました。

@kawacchu kawacchu merged commit 430a857 into master Dec 4, 2019
@kmyk kmyk deleted the fix/625 branch January 21, 2020 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sample downloader has broken (Atcoder TDPC)
3 participants