CodeForPhilly / chime

COVID-19 Hospital Impact Model for Epidemics
https://codeforphilly.github.io/chime/
MIT License
205 stars 151 forks source link

["bug"] The current % acutity (Hosp, ICU, and Vent are error prone) #336

Open Mdraugelis opened 4 years ago

Mdraugelis commented 4 years ago

The current parameters are dependent but the promoted input and use don't keep that constraint.

One could put int a hirer Ventilated rate the ICU or Hosptial, but this is not actually possible.

Furthermore, publications report rates of ICU (Critica Care) as rate hospitalizations. And rate of ventilated patients as rate of critical care patients

Fix

These fields should be updated to: Hosp %(total infections) ICU (% total hosp) Vent (% critical care)

The code should be updated to translate those numbers into %'s of total infections.

For example Hosp %(total infections) = 2.5 ICU (% total hosp) = 30% Vent (% critical care) = 66%

ICU % Total infections = Hosp %(total infections) * ICU (% total hosp)
Vent % Total infections = ICU % Total infections * Vent (% critical care)

Where 
ICU % Total infections = 0.75
Vent % Total infections = 0.495

Expected behavior:

What I got instead:

(screenshots if applicable)

gauribapat commented 4 years ago

Took a look at this, can I be assigned to this issue?

Mdraugelis commented 4 years ago

See this spreadsheet as an example of these updates. https://docs.google.com/spreadsheets/d/1hZ5HWbPqyZPLwsrc303k-Kr7Ph-YcEmjl150fQeDAzA/edit?usp=sharing

BrianThomasRoss commented 4 years ago

@jlubken any movement on this?