CodeForPhilly / CodeForPhilly/chime

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

Open
#336 3 comments 0 reactions 0 assignees View on GitHub
bug help wanted hospital-request models python
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.