Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor Compartment Display #316

Open
NXXR opened this issue Oct 20, 2023 · 3 comments
Open

Refactor Compartment Display #316

NXXR opened this issue Oct 20, 2023 · 3 comments
Labels
class:refactoring FEEDBACK Feedback on ESID question Further information is requested type:web-frontend

Comments

@NXXR
Copy link
Collaborator

NXXR commented Oct 20, 2023

Based on user feedback we need to refactor the display of compartments.

Some may not need to be displayed at all (i.e. asymptomatic).
Others may need to be aggregated into a meta compartment (e.g. infected = inf + hosp + icu + ...).

@lenarothenhaeusler @annawendler

@NXXR NXXR added question Further information is requested type:web-frontend class:refactoring FEEDBACK Feedback on ESID labels Oct 20, 2023
@NXXR NXXR added this to ESID Oct 20, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in ESID Oct 20, 2023
@lenarothenhaeusler
Copy link
Collaborator

lenarothenhaeusler commented Oct 20, 2023

The feedback from the Pilot-LHAs regarding the infectious status already displayed was:

  • The total number of infected persons should be presented (i.e. not only broken down by mildly infected, hospitalized, etc.).
  • The incidences, i.e. new infections, new hospitalizations and so on would also be interesting for the physicians
  • It would also be interesting for LHAs if hospitalized could be broken down into "hospitalized for the disease (e.g., Covid-19)" or "hospitalized for another disease"

@annawendler
Copy link
Contributor

Regarding the first point: I can adapt the scripts so that all individuals that are infected (i.e. all individuals in Exposed, InfectedNoSymptoms, InfectedSymptoms, InfectedSevere (Hospitalized) and InfectedCritical (ICU)) are aggregated into one compartment. Should we call this 'Infected' / 'Infiziert' for now or is there a better name? Do you know if there is any interest in displaying the Dead compartment as well?

@NXXR
Copy link
Collaborator Author

NXXR commented Nov 8, 2023

There were a few messages on mattermost on this topic.
I'll add a summary here so we have everything in one place:

  • The total number of infected persons should be presented (i.e. not only broken down by mildly infected, hospitalized, etc.).

We came to the conclusion that the most versatile way forward would be to keep the specific compartments data-wise in the backend and aggregate the relevant compartments to be displayed as one "infected" compartment in the frontend.

  • The incidences, i.e. new infections, new hospitalizations and so on would also be interesting for the physicians

@mknaranja said he wants to switch [the reporting] to incidence numbers instead of prevalent numbers sooner or later.

  • It would also be interesting for LHAs if hospitalized could be broken down into "hospitalized for the disease (e.g., Covid-19)" or "hospitalized for another disease"

@mknaranja clarified that the model only reports hospitalized numbers in regards to the modeled disease.
Total hospitalizations (including unrelated hospitalized) cannot be displayed at the moment because we do not have relevant data inputs (i.e. total hospitalization numbers for districts).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
class:refactoring FEEDBACK Feedback on ESID question Further information is requested type:web-frontend
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants