CodeForPhilly / CodeForPhilly/chime
["bug"] The current % acutity (Hosp, ICU, and Vent are error prone)
- Lenguaje dominante
- Python
- Estrellas
- 210
- Forks
- 153
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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)
Guía de contribución
Línea de trabajo
Comienza localizando los puntos de entrada del modelo que consumen las entradas Hospitalization, ICU y Ventilated; en el issue no se nombra ningún archivo ni prueba. Comprueba cómo se interpretan actualmente estos valores y, después, verifica que los porcentajes de ICU y Vent se traduzcan a partir de las tasas dependientes indicadas y conserven sus restricciones en el cálculo de ejemplo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- backend
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100