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

「陽性患者の属性」と「陽性患者数」の件数 #58

Closed
hidamasaki opened this issue Apr 24, 2020 · 7 comments
Closed

「陽性患者の属性」と「陽性患者数」の件数 #58

hidamasaki opened this issue Apr 24, 2020 · 7 comments
Assignees
Labels
bug Something isn't working hotfix

Comments

@hidamasaki
Copy link

起こっている問題 / The Problem

  • 「陽性患者の属性」と「陽性患者数」の件数が更新されない。
  • 4/24時点で新たな陽性患者の情報をアップロード(累計66名)したが、上記に反映されない。

スクリーンショット / Screenshot

陽性患者数等

期待する見せ方・挙動 / Expected Behavior

  • 「陽性患者の属性」と「陽性患者数」の件数が正しく(「検査陽性者の状況」の「陽性者数(累積)」の数字と合致するようにしたい。

動作環境・ブラウザ / Environment

  • macOS / Windows / Linux / iOS / Android
  • Chrome / Safari / Firefox / Edge / Internet Explorer
@hidamasaki hidamasaki added the bug Something isn't working label Apr 24, 2020
@donuzium
Copy link
Collaborator

おそらく原因は、ファイル更新日の前日まで時間軸(x軸)を作るという仕様にしたからだと思います。

hotfixとして扱いますので、すぐに対応します。

@donuzium donuzium self-assigned this Apr 24, 2020
@hidamasaki
Copy link
Author

@donuzium さん

ありがとうございます。
今後ですが、把握できる分は当日中に更新することとなります。

@donuzium
Copy link
Collaborator

donuzium commented Apr 24, 2020

何を最終日にするのかと言うのは難しいところですが、陽性患者数に関しては以下の優先順位最終日を決める優先順位で対応したいと思いますがいかがでしょうか @hidamasaki

※いずれも陽性患者数・属性のみに関連します。

  • データの最終日はファイル更新時間の12時間前とする。
    • このことで、午前中にアップがされれば前日までの分、午後にアップされれば当日の分として最終日を算出することになる。
  • いずれも公表日が上記の条件より新しくなる事があれば(あまりケースは想定できないですが、未来の予定を入れる操作をしたようなケースに限られると思います)、その公表日を最終日とする。

上記の条件で、当面の運用は問題ないんじゃないかと思っています。
ただし明示的に「最終公表日がいつか?」という情報があることが正確だと思います。
この件は追ってIssue化していきましょう。

@donuzium
Copy link
Collaborator

優先順位 => 最終日を決めるための優先順位 についてですのでコメント修正しました。

@donuzium
Copy link
Collaborator

とりいそぎ、上記の修正を入れて書き出しをしてみました。
データの書き出しはデプロイと連動していないので、即時反映されています。

Screen Shot 2020-04-24 at 23 03 50

@hidamasaki
Copy link
Author

@donuzium さん

迅速な御対応、ありがとうございます!
最終日の決め方については、御提案のとおりで問題ないものと思っております。
最終公表日の明示については追って議論させてください。

@donuzium
Copy link
Collaborator

では残りの課題を #59 に引き継ぎ、解決したコードを取り込んだため、クローズ致します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hotfix
Projects
None yet
Development

No branches or pull requests

2 participants