-
Notifications
You must be signed in to change notification settings - Fork 20
Description
Feature description
Quoting @DavidKerkmann from #681
We remove the InfectionStates Exposed and Recovered from the ABM. Removing Recovered was already planned, as there is no difference to Susceptible if we only look at the symptoms. Removing Exposed leads to every newly infected person moving to InfectedNoSymptoms immediately and they do have a chance to infect people as of the current infectivity model.
Adding a threshold for being able to infect others is an option, but I commend that this should be handled in addition to this change to account for the needs of comparison to other models or the needs of any external party. In this regard, I suggest moving the infectivity function to the parameters, so it can be set individually for any virus (type).
Going further, an idea would be to rename InfectionState to SymptomaticState or similar, and considering renaming of some of the attributes too, as there is no complete 1:1 relation to the InfectionStates of the ODE anyway. For example, we could have NoSymptoms, InfectedNoSymptoms, InfectedSymptoms, InfectedSevere, InfectedCritical, Dead. Looking at this, we can even combine the first two states, as we would only track symptoms with this.
Additional context
No response
Checklist
- Attached labels, especially loc:: or model:: labels.
- Linked to project.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status