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

Alle Tendenz-Pfeile seit letztem Update grün und nach unten zeigend... #86

Closed medicus07 closed 3 years ago

medicus07 commented 3 years ago

Stimmt aber nicht... habe es mehrfach beobachtet. Inzidenz ist nach z.B Wochenende wieder gestiegen...

firebowl commented 3 years ago

E1F72068-30A6-43F4-BFF0-04391847513C Definitiv nicht grün

rphl commented 3 years ago

Bei mir auch nicht. Poste bitte deine WidgetParameter oder groben Standort. Dann könnte ich es prüfen image

medicus07 commented 3 years ago

Hmmm... komisch.

0,49.459,8.170,Bad Dürkheim;1,49.346,8.150,Neustadt

Bildschirmfoto 2020-12-02 um 19 51 43

altes und neues Widget, selbe Parameter...

rphl commented 3 years ago

So wie es aussieht fallen bei dir die Werte oft... Was die vielen grünen Pfeile erklären würde.

Habe mal eine Option hinzugefügt, mit der man sich die Live berechneten Inzidenzen der letzten Tage anzeigen lassen kann. Sieht für mich plausiblel aus.

2020-12-02 20:16:06: "calcIncidence" | "s0"
2020-12-02 20:16:06: "02.12.2020" | 106.27793564531812 << fällt
2020-12-02 20:16:06: "01.12.2020" | 115.32286633853668 << steigt
2020-12-02 20:16:06: "30.11.2020" | 107.78542409418787 << fällt
2020-12-02 20:16:06: "29.11.2020" | 119.09158746071108 << fällt
2020-12-02 20:16:06: "28.11.2020" | 121.35282013401572 << steigt
2020-12-02 20:16:06: "27.11.2020" | 119.09158746071108 << fällt
2020-12-02 20:16:06: "26.11.2020" | 128.89026237836453 << fällt
2020-12-02 20:16:06: "25.11.2020" | 136.42770462271332 << fällt
2020-12-02 20:16:06: "24.11.2020" | 143.21140264262726  << fällt
2020-12-02 20:16:06: "23.11.2020" | 145.4726353159319 << steigt
2020-12-02 20:16:06: "22.11.2020" | 134.92021617384356
....

Sehe das Alte Widget hat Lücken unten für heute/gestern. Somit fehlen Werte.

medicus07 commented 3 years ago

Ok... Auch gestern war der Pfeil grün, obwohl er am 1.12. hätte rot/steigend sein müssen. Ich berichte morgen noch mal... Werte für morgen hab ich schon 109 (DÜW) und 126 (NW). Also erster Pfeil steigend...

rphl commented 3 years ago

Glaube das könnte auch an Zeitverzögerten Meldungen der Fälle liegen. Die werden ja immer Live aus der API gezogen.

Sollten also für einen vorhergehenden Zeitraum die Fälle sich ändern (MeldeDatum beim Gesundheitsamt ist entscheidend), würde sich ja auch die Inzidenz ändern.

Ich werde mal auch die Fälle in console darstellen. Dann kann man es noch besser vergleichen.

medicus07 commented 3 years ago

Same today... der eine Pfeil hätte rot sein müssen.

IMG_3269

medicus07 commented 3 years ago

Das ist mir klar... aber die 109 sind ja der 7-Tageswert und gestern lag dieser bei 106, also heute höher... ich beziehe mich ja nicht auf die absoluten Zahlen.

dennerforen commented 3 years ago

Wenn die Daten live ausgewertet werden, macht ein Vergleich mit alten Zahlen keinen Sinn. Es ist doch ein dynamischer Vorgang mit nach Meldungen, Korrekturen und Nachmeldungen.

1rNidiD commented 3 years ago

Ok jetzt habe ich mal bei mir geschaut, habe auch den selben Fehler..

Gestern: image

Heute: image

Meine Parameter: 0,48.1351,11.5819,Stadt München;1,48.0782,11.6392, Neubiberg

Der Grüne Pfeil bei Stadt München, Bayern und Deutschland müsste eig rot sein..

medicus07 commented 3 years ago

IMG_3270

Es müsste bei mir heute auch alles rot sein...

mmattersdorfer commented 3 years ago

Im Landkreis Ludwigsburg (0,48.917605,9.223896) müsste der Pfeil heute auch rot sein. Er ist aber grün.

tomacco81 commented 3 years ago

Bei mir ist es auch wieder so, dass der Trendpfeil für den LK nach unten zeigt, obwohl der Wert gestiegen ist (gestern 202, heute 212). Ich habe mir daraufhin das debug-Log näher angeschaut und festgestellt, dass der gestrige Wert im Log viel höher ist, als das Widget angezeigt hat (gestern 234, heute 212). Ich kapiere nicht warum der Wert rückwirkend nach oben korrigiert wurde. Die Historie wird doch vom Widget erstellt und nicht neu abgerufen, daher dürfte der Wert ja nicht rückwirkend verändert worden sein oder liege ich da falsch? Gestern haben die Werte im Debug-Log jedenfalls noch gepasst...

medicus07 commented 3 years ago

Bei mir sind alle Pfeile nur noch grün und das seit vielen Tagen, einen Wechsel hat es nicht mehr gegeben. Irgendwie scheinen im Hintergrund die Zahlen wirklich falsch zu sein.

tomacco81 commented 3 years ago

Ich glaube das Problem gefunden zu haben. Das Widget ruft zum einen den aktuellen Tageswert vom RKI ab. Außerdem werden in der neuen Widget-Version auch die Historiendaten vom RKI geladen. Wenn das RKI nun die Historiendaten rückwirkend nach oben korrigiert, tritt das Problem mit dem Trendpfeil auf... Ich habe bei mir eben verschiedene Stände der json Datei (Landkreis) verglichen und tatsächlich unterschiede in den Historienwerten festgestellt. Das erklärt auch warum bei mir heute der Inzidenzwert für den Vortag höher und somit der Pfeil grün ist.

rphl commented 3 years ago

Das ist leider das Problem, was ich bereits erklärt habe. Je nachdem wieviel das RKI Nachträgt/Korrigiert ändert sich eben die Inzidenz (Was eben Korrekt ist).

Hier stellt sich die Frage wie man trotz des verspäteten Updates in einigen Regionen, man valide Trendpfeile anzeigen lassen kann. Option1: Evtl. muss man hier gestern <> vorgestern vergleichen und auf den Tagesaktuellen Inzidenzwert verzichten. Option2: Sowas wie Wochentag (Gestern) <> Wochentag (Gestern) Vorwoche. Option3 Wochendurchschnitt vergleichen? Aktuelle <> Vorwoche?

rphl commented 3 years ago

@mmattersdorfer

Das sind die Inzidenzwerte für deinen Standort (Kann via debugIncidenceCalc angezeigt werden). Grün ist wohl schon Richtig.. Auch wenn man gestern <> vorgestern vergleicht. Hmm.

2020-12-10 20:04:18: "10.12.2020" | 201 | 130.72422688445482
2020-12-10 20:04:18: "09.12.2020" | 102 | 145.5750857591264
2020-12-10 20:04:18: "08.12.2020" | 117 | 153.8255629117217
2020-12-10 20:04:18: "07.12.2020" | 83 | 160.0592567603493
2020-12-10 20:04:18: "06.12.2020" | 15 | 158.2258173931059
tomacco81 commented 3 years ago

Das ist leider das Problem, was ich bereits erklärt habe. Je nachdem wieviel das RKI Nachträgt/Korrigiert ändert sich eben die Inzidenz (Was eben Korrekt ist).

Du hast völlig recht, und die Berechnung und die Trendpfeile passen in dem Fall. Mir war nur nicht klar, dass das RKI rückwirkend teilweise so viel verändert. Bringt ja auch nix wenn sich durch das Nachtragen der Inzidenzwert der Vergangenheit verändert - die Zahlen wurden ja bereits so an die Bevölkerung kommuniziert und rückwirkend schaut sich das keiner mehr an. Bestes Beispiel: Das RKI kommuniziert am 08.12.2020 für den LK Rottweil einen Wert von 186,6. Und wenn man das ganze jetzt betrachtet lag der Wert eigentlich bei 234,5 🙈

rphl commented 3 years ago

@tomacco81 Hier sind die Regionen sehr Unterschiedlich... Finde es auch seeeeehr suboptimal. Die Berichte sind auch nur eine Momentaufnahme, und das Widget macht gerade irgendwie ein Spagat...

Naja, habe deswegen erstmal den Trend auf Wochenbasis (Option3) umgebaut. Finde es aktuell so robuster, auch für die Regionen die viel Nachpflegen. Habe dementsprechend die README aktualsiert.

Bildschirmfoto 2020-12-10 um 20 55 42
tomacco81 commented 3 years ago

Ich denke das mit dem Wochendurchschnitt trifft es ganz gut. Super, dass du es so schnell anpassen konntest!