Test Coronavirus Statistics for ioBroker
-
Danke. Aber kurze Frage. Wieso verwendest du nicht die Quelle, die ich hier am Ende angegeben habe?
Also gibts einen Grund, oder einfach nur zu viel Aufwand?
Bei Letzterem würde ich mir das nämlich ggf. mal anschauen. Wollte sowieso mal ein bisschen in die Adapterentwicklung einsteigen, falls ich die Zeit finde.https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/issues/238
closed "Impfquote" values are 'null %' #238
-
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Also gibts einen Grund, oder einfach nur zu viel Aufwand?
das letztere da ich dafür die komplette logic wieder umbauen muss wie Impfdaten eingelesen werden und dazu fehlt mir leider gerade die zeit
Wir verwenden die officiële API (die auch diese git Seite wieder nimmt) und die Datei vom RKI sind also selben daten nur mit weniger schritten/Abhängigkeiten dazwischen
-
@dutchman
Ok verstanden.
Würdest du den PR mergen, wenn ich den mache, oder willst du die Abhängigkeit nicht?
Wären dann halt unabhängig von solchen Aktualisierung. Aber natürlich abhängig von Schnelligkeit und Fortbestand des Repos. -
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Würdest du den PR mergen, wenn ich den mache, oder willst du die Abhängigkeit nicht?
klar why not
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Wären dann halt unabhängig von solchen Aktualisierung. Aber natürlich abhängig von Schnelligkeit und Fortbestand des Repos.
Das isst der Grund warum ich es bei den offiziellen API's und RKI daten belasse
-
@dutchman
Ich fände es als Developer halt ätzend auf unstrukturierte Daten aufzusetzen.
Hätte keinen Bock immer alles upzudaten, nur weil ein DAU der Meinung ist, die Spalten anpassen zu müssen
-
@dutchman said in Test Coronavirus Statistics for ioBroker:
geschafft, es gibt in der Excel Datei keine details mehr bezueglich welchem typen die daten sind da auch nicht mehr im adapter vorhanden und Datenpunkte werden, wen aktiviert, geloescht
0.7.0-5 (2021-06-25)
(DutchmanNL) Bugfix: State attribute definition missing for + undefined
(DutchmanNL) Bugfix: Vaccination data for Germany, several states missing in new API and will not be updated anymore !
(DutchmanNL) Bugfix: State value to set for "xxx.BL" has to be type "number" but received type "string", solves #237
(DutchmanNL) Bugfix: State value to set for "xxx.Tests*" has to be type "string" but received type "number", solves #237Bei mir funzt die 0.7.0-5 btw nicht:
-
@Dutchman
liegt denke ich daran, dass meine federalStateName undefined o.ä. ist. Ich muss mich erst noch in deinen Adapter reinfuxen. Aber Ich denke da gehört "Gesamt" statt federalStateName hin...?
Der Adapter bietet ja gar keine Quoten pro Bundesland, oder?
-
@Dutchman
Ok, sorry. War in der falschen Funktion .
Hab deinen Adapter jetzt grob verstanden. Zumindest die Impfungen.
Mein Fehler muss in Zeile 498ff. behoben werden. Die Spaltennamen sind falsch.
Muss jetzt leider los, aber kann nächste Woche nen PR stellen.
Vllt. mach ich mich dann auch mal an die Umstellung auf das andere Repo.
-
@dutchman said in Test Coronavirus Statistics for ioBroker:
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Würdest du den PR mergen, wenn ich den mache, oder willst du die Abhängigkeit nicht?
klar why not
Hab jetzt zwei PRs gestellt.
Der erste fixt die Spalten Namen für deine aktuelle Implementierung der Impfdaten.Beim zweiten PR hätte ich gerne erstmal deine Meinung. Ich kann coden, aber ich hab wenig github und iobroker Modul Erfahrung
Ich hab die Source jetzt umgebaut und getestet. Funktioniert alles. Mit der Implementierung sind wir natürlich gerade noch langsamer unterwegs, da von zwei Quellen die Daten geladen werden (Excel und Github). Kann ich mir im nächsten Schritt dann aber weiter anschauen.Freu mich wenn du mir kurz Rückmeldung gibst.
-
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
@Dutchman
Ok, sorry. War in der falschen Funktion .
Hab deinen Adapter jetzt grob verstanden. Zumindest die Impfungen.
Mein Fehler muss in Zeile 498ff. behoben werden. Die Spaltennamen sind falsch.
Muss jetzt leider los, aber kann nächste Woche nen PR stellen.
Vllt. mach ich mich dann auch mal an die Umstellung auf das andere Repo.danke ! mist den hab ich übersehen fuer die Gesamtzahlen ich mache dan mal die final version mit dem PR
Den 2ten PR schaue ich mir dan die Woche in ruhe an, danke dafuer !
-
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Beim zweiten PR hätte ich gerne erstmal deine Meinung. Ich kann coden, aber ich hab wenig github und iobroker Modul Erfahrung
habe auf git kommentiert/reviewd
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Ich hab die Source jetzt umgebaut und getestet. Funktioniert alles. Mit der Implementierung sind wir natürlich gerade noch langsamer unterwegs, da von zwei Quellen die Daten geladen werden (Excel und Github). Kann ich mir im nächsten Schritt dann aber weiter anschauen.
hmm interessant bei mir kan der code so nicht klappen schau bitte nochmal drüber
Die excel Datei koennen wir nur von weck wen die Bundesländer auch ueber die GitHub API kommen. -
@dutchman said in Test Coronavirus Statistics for ioBroker:
Die excel Datei koennen wir nur von weck wen die Bundesländer auch ueber die GitHub API kommen.
Das stimmt. Aber jetzt sind zumindest mal die Gesamt-Impfungsdaten relativ stabil.
Mich persönlich interessieren die Impfdaten für Bundesländer nicht. Die orientieren sich ja eh stark an den Impfdaten von Deutschland und haben keinen konkreten Einfluss auf Lockdown-Regeln o.ä. Einschränkungen.
Ich würde sie daher rausschmeißen, wenns mein Repo wäre Aber wir können sie auch drin lassen, wenn da jemand ab und zu die Struktur bei Excel-Änderungen anpasst. -
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
Ich würde sie daher rausschmeißen, wenns mein Repo wäre Aber wir können sie auch drin lassen, wenn da jemand ab und zu die Struktur bei Excel-Änderungen anpasst.
ich weis das es einige gibt die es gerne per Bundesland hätten also lassen wir das mal drin
Naester schritt waehre, wie auch auf git genannt :P, die daten fuer alle Länder auf zu bereiten fuer Impfungen -
da ist jetzt beim mergen irgendwas komplett durcheinander gekommen, kannst du bitte nochmal nen `PR machen mit diesem code teil irgendwie wurde da der alte code übernommen
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/blob/main/main.js#L351-L375
-
@dutchman said in Test Coronavirus Statistics for ioBroker:
da ist jetzt beim mergen irgendwas komplett durcheinander gekommen, kannst du bitte nochmal nen `PR machen mit diesem code teil irgendwie wurde da der alte code übernommen
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/blob/main/main.js#L351-L375
Hotfix PR hab ich gestellt.
Zusätzlich hab ich noch die Impfdaten für alle Länder als extra PR angelegt. Der ist aber etwas eskaliert
Viel Spaß beim reviewen, lul -
@jlssmt sagte in Test Coronavirus Statistics for ioBroker:
@dutchman said in Test Coronavirus Statistics for ioBroker:
da ist jetzt beim mergen irgendwas komplett durcheinander gekommen, kannst du bitte nochmal nen `PR machen mit diesem code teil irgendwie wurde da der alte code übernommen
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/blob/main/main.js#L351-L375
Hotfix PR hab ich gestellt.
Zusätzlich hab ich noch die Impfdaten für alle Länder als extra PR angelegt. Der ist aber etwas eskaliert
Viel Spaß beim reviewen, lulCool danke!
-
Hallo,
gibt es eigentlich den 7 Tages Inzidenzwert auch für andere Länder?
(z.B. Spanien, Türkei, etc.) -
Das würde mich auch interessieren Ich habe heute in der Türkei einen Inzidentswert von 94 gehabt aber im Iobroker zeigt er mir 6,37 an bei kritisch für 1000000 einwohner vielleicht kann mir das einer erklären
-
@Dutchman
wie siehts mit den PRs aus?
Insbesondere der Hotfix wäre wichtig. -
Hallo,
gibt es im Adapter die 7-Tage Inzidenz für Deutschland?
Habe dafür immer das "kritisch für 1000000" genommen. Entspricht das der Inzidenz?Das steht jetzt bei mir seit Tagen aber immer auf 4,2.
Das haben wir ja leider schon länger nicht mehr.....
