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

stats.csv: rename cases.active.todate to cases.active #11

Closed
lukarenko opened this issue Apr 28, 2020 · 3 comments
Closed

stats.csv: rename cases.active.todate to cases.active #11

lukarenko opened this issue Apr 28, 2020 · 3 comments
Assignees

Comments

@lukarenko
Copy link
Contributor

lukarenko commented Apr 28, 2020

We have now added estimated recovered, closed (recovered+deceased) and active cases.

  • cases.confirmed.todate
  • cases.recovered.todate = cases.confirmed.todate(today) - cases.confirmed.todate(-21days)
  • cases.closed.todate = cases.recovered.todate + state.deceased.todate
  • cases.active = cases.confirmed.todate - cases.closed.todate

As cases.active is is current (today's) state, we should remove wrong .todate suffix.

Note: as this fields were not used much before, we do not expect any breakage.

@lukarenko lukarenko self-assigned this Apr 28, 2020
@lukarenko
Copy link
Contributor Author

@MihaMarkic, we need to rename it first in REST.

@mkadunc
Copy link
Contributor

mkadunc commented Apr 28, 2020

@lukarenko cases.closed.todate je narobe v tekstu zgoraj. Moralo bi bit closed.todate = recovered.todate + deceased.todate
(Predlagam da kar popravis comment.)

Se komentar - invariante za active (ne vem, če kaj pomaga):

recovered.todate === recovered.active
deceased.todate === deceased.active
closed.todate === closed.active
closed.todate === recovered.todate + deceased.todate
confirmed.todate === recovered.active + confirmed.active + deceased.active
hospital.active = hospital_regular.active + icu.active
confirmed.active = nonhosp.active + hospital_regular.active + icu.active

@lukarenko
Copy link
Contributor Author

Old cases.closed.todate removed

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

No branches or pull requests

2 participants