search

br-data / corona-api

Einheitliche API für alle Corona-Daten: Neuinfektionen, Todesfälle, Inzidenzen, Impfquoten, Hospitalisierungen, Intensivpatienten und mehr
0 stars 0 forks source link

Updaten auf neue API #43

Open looreen opened 2 years ago

looreen commented 2 years ago

Die beiden Grafiken bauen auf Daten im Corona Data Workers Repo auf, die auf einer Virtual Machine laufen, die Steffen gern aussortieren möchte. Daher sollen die auf die neue API plus Spreadsheet umgestellt werden.

https://datawrapper.dwcdn.net/NWcZ8 https://datawrapper.dwcdn.net/XiHIr

looreen commented 2 years ago

Für Bayern unterscheiden sich die Angaben zur Bevölkerung je Altersgruppe leicht im Vergleich zu dem, was in der API hinterlegt ist. Siehe Screenshot: Oben sind die Daten aus der API, unten die aus den Corona Data Workers. Bildschirmfoto 2022-10-19 um 12 03 44

In den berechneten Inzidenzen kommt es daher zu leichten Verschiebungen: In der Altersgruppe 0 bis 4 für Woche bis 17.10. etwa 153 statt bisher 154. In der Gruppe 60 bis 79 von 1015 auf 1010. Ist das problematisch? @stekhn

stekhn commented 2 years ago

Grundsätzlich sehe ich darin kein großes Problem. Ich frage mich aber, woran das liegt. Verwenden wir hier möglicherweise zwei unterschiedliche Bevölkerungsstände? Wenn ja, sollten wir die aktuellen Alterskohorten verwenden. Problematisch wäre, wenn die Unterschiede durch einen Berechnungsfehler zustande kommen.

looreen commented 2 years ago

Bildschirmfoto 2022-10-20 um 11 53 28 Es werden unterschiedliche Zeitpunkte der Bevölkerungszahlen verwendet. Michael hat den Stand vom 31.12.2019 benutzt, die Corona API nutzt die vom 31.12.2020. Beide unterscheiden sich naturgemäß zu denen vom 31.12.2021. Ich würde vorschlagen, wir nutzen die der Corona API (2020) und machen ein neues Issue auf, die Bevölkerungszahlen für alle BL, nicht nur Bayern, in der Corona API auf den Stand von 2021 anzupassen. Da können wir dann auch Fragen wie mögliche Verzerrungen berücksichtigen. Passt das für dich @stekhn ?

stekhn commented 2 years ago

Das halte ich für eine sinnvolle Lösung. Vielen Dank für die Recherche, @looreen 🙏

looreen commented 2 years ago

Hier ist ein Spreadsheet, das direkt auf die Corona API zugreift und die gleichen Daten erarbeiten sollte. https://docs.google.com/spreadsheets/d/1-UcAl2QuwQSQmvoy2FbsrXfrKmHv6uzxsAnam_u256M/edit#gid=2074220014 Das sollte dieses Problem lösen. @marco-le Kannst du das Sheet durchgehen und wenn alles passt, den richtigen Link in der Originalgrafik ersetzen?

marco-le commented 2 years ago

Die Originalgrafik Bayern ist umgestellt. @looreen: Gibt es das auch noch für Deutschland, oder muss das noch erstellt werden?

looreen commented 2 years ago

Gibt es bisher nur für Bayern, Deutschland kann aber natürlich analog erstellt werden.

Marco Lettner @.***> schrieb am Mi., 2. Nov. 2022, 07:44:

Die Originalgrafik Bayern ist umgestellt. @looreen https://github.com/looreen: Gibt es das auch noch für Deutschland, oder muss das noch erstellt werden?

— Reply to this email directly, view it on GitHub https://github.com/br-data/corona-api/issues/43#issuecomment-1299654702, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJPUAOBZWUVGGLF3AECCH2LWGIETLANCNFSM6AAAAAARIEW4II . You are receiving this because you were mentioned.Message ID: @.***>

looreen commented 2 years ago

Ich hab in dem gleichen Spreadsheet ein äquivalentes Spreadsheet für Deutschland erstellt und den Link in Datawrapper geändert. Damit sollten die Grafiken fertig sein und keine mehr auf die Dataworkers zugreifen.