hrsano645 / covid19-gen-datajson-shizuokapref

静岡県版 新型コロナウイルス対策サイトのdata.jsonを生成するスクリプト
1 stars 3 forks source link

patients_summary(陽性患者数カード)で更新がない日の項目が追加されてしまう #13

Open hrsano645 opened 4 years ago

hrsano645 commented 4 years ago

問題の詳細

https://github.com/hrsano645/covid19-gen-datajson-shizuokapref/issues/9 の対応中に気が付いた問題

slackにて@aktnk を引用

8/2 23時の時点で 1)オープンデータの _patients.csvには7/31までのデータ _test_number.csvには7/30までのデータ _call_center.csvには7/30までのデータ が記載されています。 2)そして、data.jsonには patientsには8/2までのデータあり、但し、7/31までのデータ+8/1&2は0としてデータが追加されている。 querentsには7/30までのデータ inspections_summaryには7/30までのデータ 3)最後に対策サイトでは 陽性患者数のグラフは8/2まで 相談件数のグラフは7/30まで 検査実施件数のグラフは7/30まで となっている。 ⇒patientsに関するデータの扱いが、他とは違い、更新した日まででCSVにない場合は、0としてデータが補間されている。だから、このグラフだけ0と表示される。 (編集済み)

hrsano645 commented 4 years ago

対応方法検討しました。(面倒なので、自分のslack上のメンションをコピペ)

patientsには8/2までのデータあり、但し、7/31までのデータ+8/1&2は0としてデータが追加されている。

これはdata.json>patients_summary(陽性患者数)のことだと思います。ここの生成は、日付のラベルを生成するときに、今日の日付を最終日にしています。 本来だとどのデータも7/31までですが、この部分だけ更新日と同じ日付のデータが生成されてますね。修正どうしようかな:考え中: 処理を考えると、patientsの日付事の行数にすればよかったりするので、すぐにできるかな。 日付事のグルーピングを行って、カウントすればよいはず この件issueにしておきます。 グルーピングしても、patientsに出現しない日付を入れないといけないからちょっと面倒で、8/5以降に対応したいですね

hrsano645 commented 4 years ago

対応方法まとめ

hrsano645 commented 3 years ago

まだ修正されていないが、オープンデータ更新時のissue https://github.com/aktnk/covid19/issues/68 に関連するので、そこに合わせて修正します。

patient_summary -> 陽性患者数のカードの現状をまとめると

オープンデータ更新日と整合性を合わせる必要があるため、このような変更予定です

hrsano645 commented 3 years ago

関連issue https://github.com/hrsano645/covid19-gen-datajson-shizuokapref/issues/24