search

TB-DevAcc / MVSEIQRD_Model

A simulation of the coronavirus pandemic in Germany based on the SEIR model.
MIT License
1 stars 0 forks source link

Load default_values of class I as relative to N #56

Closed Sp3zif1sch closed 2 years ago

Sp3zif1sch commented 2 years ago

Description Die Werte von Klasse I in default_values sollten wir evtl. auf relative Werte ändern, da wir so unabhängig von J wären.

Is your feature request related to a problem? Please describe. Es werden von den Stammdaten 401 Landkreise mit Einwohnern N pro Altersgruppe eingelesen, zu denen jeweils ein passender Anfangswert in I existieren muss, da sonst die Simulation sehr merkwürdige Dinge tut bzw. gar nicht läuft.

Describe the solution you'd like Die Startwerte in default_values von I auf relative Werte pro Altersgruppe setzen, welche dann entsprechend mit N multipliziert und in I gespeichert werden, sodass unabhängig von der Anzahl J immer passende Werte in I zum Start der Simulation vorhanden sind.
Alternativ könnte man auch eine zweite Klasse I_0 verwenden, die diese relativen Werte zu N bereitstellt.

Eure Meinung?

TB-DevAcc commented 2 years ago

Ja bin ich für alle Klassen dafür (alles relative Werte im geschlossenen Intervall [0, 1]). Haben wir glaube ich auch schon irgendwo mal besprochen (ist nämlich für die Sanity checks auch sehr viel besser).

TB-DevAcc commented 2 years ago

Ah, ist das nicht quasi #28 ?

Sp3zif1sch commented 2 years ago

Ah, ist das nicht quasi #28 ?

War #28 nicht auf die Simulation bezogen? (Habe die Simulation mit relativen Werten bis jetzt nur kurz getestet und da kamen sehr merkwürdige Werte raus)

Wenn es bei #28 auch "nur" um die default_values geht, dann lösche ich den Issue hier und wir machen in #28 weiter.

TB-DevAcc commented 2 years ago

Für mich gehört das eigentlich zusammen, wobei simulator.py das auch gerne handhaben kann wie es will. Wichtiger wären mir aber die default_values

Sp3zif1sch commented 2 years ago

Werte für I können durch #57 als relative Werte eingelesen werden.