search

rphl / corona-widget

COVID-19 Widget ( Inzidenz / Impfquoten / Hospitalisierungen ) für iOS innerhalb Deutschlands 🇩🇪 (Kreis/Stadt + Bundesland + Trend)
Other
648 stars 95 forks source link

Pfeil von Inzidenzwert auf Lkr-Ebene falsch #97

Closed EscolarProgramming closed 3 years ago

EscolarProgramming commented 3 years ago

Hallo, Leidet scheint der Pfeil, der die Tendenz des Inzidenzwerts auf Landkreisebene angibt, wieder nicht zu stimmen. Beispiel Lkr Rottal-Inn: Vorgestern 150, gestern 184, heute 201 - trotzdem ist der Pfeil immer noch grün und zeigt nach unten.

Wäre schön, wenn das gefixxt wird :)

rphl commented 3 years ago

Hallo. Der Trend ist seit neuesten ein Wochentrend. Siehe auch Readme :-)

Viele Regionen melden oft verspätet, und auch für vergangene Zeiträume. Somit ändert sich unweigerlich auch die Inzidenz der vorhergehenden Tage. Das macht die Pfeile unplausibel.

Deswegen sind die Trends jetzt Wochenbasierd. D.h Durchschnitt Woche (1-7Tage) <> Durchschnitt Vorwoche (8-14 Tage). Das macht den Trend für alle Regionen deutlich robuster gegenüber "korrekturren/updates".

EscolarProgramming commented 3 years ago

Ah, sorry, nicht mitbekommen. Macht aber Sinn

MeteoMartini commented 3 years ago

Hallo. Der Trend ist seit neuesten ein Wochentrend. Siehe auch Readme :-)

[...]

Deswegen sind die Trends jetzt Wochenbasierd. D.h Durchschnitt Woche (1-7Tage) <> Durchschnitt Vorwoche (8-14 Tage). Das macht den Trend für alle Regionen deutlich robuster gegenüber "korrekturren/updates".

Sorry, dass ich hier einhake, aber bei mir sieht der Trend auch nach Umstellung inkorrekt aus:

2020-12-14 17:00:56: "14.12.2020" | 211 | 176.275542475914 2020-12-14 17:00:56: "13.12.2020" | 46 | 168.6186256167147 2020-12-14 17:00:56: "12.12.2020" | 96 | 155.96806732760285 2020-12-14 17:00:56: "11.12.2020" | 225 | 134.32895446464835 2020-12-14 17:00:56: "10.12.2020" | 298 | 115.68602645964138 2020-12-14 17:00:56: "09.12.2020" | 71 | 122.677124461519 2020-12-14 17:00:56: "08.12.2020" | 100 | 122.17776031852775 2020-12-14 17:00:56: "07.12.2020" | 223 | 92.88173059637396 2020-12-14 17:00:56: "06.12.2020" | 0 | 125.50685460513613 2020-12-14 17:00:56: "05.12.2020" | 20 | 127.50431117710114 2020-12-14 17:00:56: "04.12.2020" | 95 | 142.6516901811693 2020-12-14 17:00:56: "03.12.2020" | 186 | 135.49413746496128 2020-12-14 17:00:56: "02.12.2020" | 113 | 140.9871430378651 2020-12-14 17:00:56: "01.12.2020" | 97 | 136.99222989393505 2020-12-14 17:00:56: "30.11.2020" | 47 | 135.8270468936221

Durchschnittliche Inzidenzwerte 142 (letzte Woche) 129 (vorvergangene Woche)

Dies müsste aktuell einen Pfeil nach oben bedeuten, sofern ich den aktuellen Code richtig deute. Tatsächlich sieht es wie folgt aus: image

rphl commented 3 years ago

Das stimmt. Ich checke es mal 😬

rphl commented 3 years ago

Habe es nochmal gecheckt, der Grund war/ist das "heute" nicht eingerechnet wird. Denn dann ergeben sich

Ak. Woche: 130,334041320718 Vorwoche: 134,99477332197

was den sinkenden Pfeil erklärt.

MeteoMartini commented 3 years ago

Danke für's Klären! 👍🏼