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

Changes by create-pull-request action #54

Merged
merged 2 commits into from
May 14, 2020
Merged

Conversation

github-actions[bot]
Copy link

Automated changes by create-pull-request GitHub action

@yuukis
Copy link
Member

yuukis commented May 12, 2020

再陽性患者がカウントされている状態。
再陽性をカウントするのが正しいのかは不明のため、一旦県サイトの「新型コロナウイルス感染症の県内における発生状況」の更新待ち
https://www.pref.yamanashi.jp/koucho/coronavirus/info_coronavirus_data.html#Cases

スクリーンショット 2020-05-13 06 05 26

@daisuke19891023
Copy link
Member

再陽性をカウントしないロジックとすることもできますが、どちらがいいでしょうか?

@yuukis
Copy link
Member

yuukis commented May 14, 2020

再陽性をカウントするのが正しいのかは不明のため、一旦県サイトの「新型コロナウイルス感染症の県内における発生状況」の更新待ち
https://www.pref.yamanashi.jp/koucho/coronavirus/info_coronavirus_data.html#Cases

県のサイトでは再陽性は数にカウントしていない(57名のまま)ため、
再陽性はカウントしないのが正しい。

@daisuke19891023

再陽性をカウントしないロジックとすることもできますが、どちらがいいでしょうか?

ありがとうございます!
再陽性をカウントしないのが正しいようですので、お手隙の時にカウントしないようロジックを変更していただけると助かります。
一旦、この PR は手動で直します。

@yuukis
Copy link
Member

yuukis commented May 14, 2020

@daisuke19891023

再陽性をカウントしないのが正しいようですので、お手隙の時にカウントしないようロジックを変更していただけると助かります。

Issue を作成しました。ご確認よろしくお願いします。
daisuke19891023/covid19-yamanashi-scraping#63

@yuukis yuukis merged commit 94db54c into development May 14, 2020
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.

2 participants