CodeForPhilly / CodeForPhilly/chime
["bug"] The current % acutity (Hosp, ICU, and Vent are error prone)
- Ngôn ngữ chính
- Python
- Star
- 210
- Fork
- 153
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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)
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách xác định các điểm vào của mô hình sử dụng các đầu vào Hospitalization, ICU và Ventilated; issue không nêu tên tệp hoặc bài kiểm thử nào. Kiểm tra cách các giá trị này hiện được diễn giải, sau đó xác minh rằng các phần trăm ICU và Vent được chuyển đổi từ các tỷ lệ phụ thuộc đã nêu và vẫn giữ nguyên các ràng buộc của chúng trong phép tính ví dụ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100