CodeForPhilly / CodeForPhilly/chime
["bug"] The current % acutity (Hosp, ICU, and Vent are error prone)
- Dominant language
- Python
- Stars
- 210
- Forks
- 153
- PR merge metrics
- No merged PRs in 30d
Description
The current parameters are dependent but the promoted input and use don't keep that constraint.
- Hospitalization %(total infections)
- ICU %(total infections)
- Ventilated %(total infections)
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)
Contributor guide
Research direction
Start by locating the model entry points that consume the Hospitalization, ICU, and Ventilated inputs; no file or test is named in the issue. Check how these values are currently interpreted, then verify that ICU and Vent percentages are translated from the stated dependent rates and preserve their constraints in the example calculation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100