CodeForPhilly / CodeForPhilly/chime
["bug"] The current % acutity (Hosp, ICU, and Vent are error prone)
- Langage dominant
- Python
- Étoiles
- 210
- Forks
- 153
- Métriques de merge des PR
- Aucune PR mergée en 30 j
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)
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par localiser les points d’entrée du modèle qui consomment les entrées Hospitalization, ICU et Ventilated ; aucun fichier ni test n’est nommé dans l’issue. Vérifiez comment ces valeurs sont actuellement interprétées, puis vérifiez que les pourcentages ICU et Vent sont convertis à partir des taux dépendants indiqués et qu’ils conservent leurs contraintes dans le calcul d’exemple.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100