Add data on # of people on ICU

valstu / korona-info

Suomen koronavirus-tartuntatilanne

MIT License

67 stars 19 forks source link

Add data on # of people on ICU #37

Closed nurpax closed 4 years ago

nurpax commented 4 years ago

The current "daily new cases" may be limited by testing capability.

Daily patients on ICU would give a more objective number on the progress of this epidemic. ICU cases most likely won't be affected by limited testing capability.

ahatala commented 4 years ago

Created an issue on this in the HS repo as well: https://github.com/HS-Datadesk/koronavirus-avoindata/issues/52

nurpax commented 4 years ago

quarian commented 4 years ago

See https://github.com/HS-Datadesk/koronavirus-avoindata/pull/55

vkomulai commented 4 years ago

I think my PR https://github.com/valstu/korona-info/pull/57 solves this one

It shows the number of people in hospital (total), in ward, and in the intensive care unit.

valstu commented 4 years ago

This is now on production, added you @vkomula to contributors list. Thank you!

valstu commented 4 years ago

Made the bar ICU chart stacked bar chart since those three bars side by side might be misleading. Green bar is yellow and red combined.

nurpax commented 4 years ago

Big thanks @vkomulai!

The stacked chart is a big improvement, thanks!

IMO the ICU cases would be better put at the bottom, not top. I consider ICU cases to be the most important metric and its trend is important. If you put ICU at the top, it's harder to see what the trend is for ICU cases. OTOH, if ICU is bottom, you can see ICU trend and you can see the overall hospitalization (ICU and normal hospital stays) trend well.

valstu commented 4 years ago

@nurpax yeah, makes sense. This should be easy fix, just changing the order of those bars on JSX. Will do that tomorrow.