Test Coronavirus Statistics for ioBroker

[Dutchman]

@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.

[jlssmt]

@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.

[Dutchman]

@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

[Dutchman]

@jlssmt

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

[jlssmt]

@dutchman said in Test Coronavirus Statistics for ioBroker:

@jlssmt

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

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@dutchman said in Test Coronavirus Statistics for ioBroker:

@jlssmt

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

Cool danke!

[stoepselinho]

Hallo,
gibt es eigentlich den 7 Tages Inzidenzwert auch für andere Länder?
(z.B. Spanien, Türkei, etc.)

[katte]

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

[jlssmt]

@Dutchman
wie siehts mit den PRs aus?
Insbesondere der Hotfix wäre wichtig.

[David G.]

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.....

[jlssmt]

@david-g
ne gibt es aktuell nicht. kritisch/1mio ist auf jeden fall was anderes.
ich kann mich da mal ransetzen, wenn meine anderen beiden PRs durch sind. da warte ich bislang noch auf die Entwickler vom adapter.

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@david-g
ne gibt es aktuell nicht. kritisch/1mio ist auf jeden fall was anderes.
ich kann mich da mal ransetzen, wenn meine anderen beiden PRs durch sind. da warte ich bislang noch auf die Entwickler vom adapter.

Schaue sie mir morgen an war im Urlaub

[Dutchman]

@jlssmt

Habe die PR's gemehrte und den version conflist gelöst es kommt dan auch ne neue release.
Leider macht RKI wieder Probleme fuer die Bundesländer das schaue ich mir heute Abend an.

Entweder fix oder ich mache das configurable/schmeisse es raus.
Die welt daten kommen auf jedenfalls einwandfrei rein dafuer danke @jlssmt !

[jlssmt]

@dutchman
schon gesehen. bitte meine Kommentare hier beachten:
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/pull/280

jlssmt created this issue in DrozmotiX/ioBroker.coronavirus-statistics

closed New vaccination source for all countries #280

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@dutchman
schon gesehen. bitte meine Kommentare hier beachten:
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/pull/280

gesehen danke!!!

[stoepselinho]

gibt es im bald im Adapter die 7-Tage Inzidenz von allen Ländern?

[jlssmt]

@stoepselinho said in Test Coronavirus Statistics for ioBroker:

gibt es im bald im Adapter die 7-Tage Inzidenz von allen Ländern?

Ich muss noch schauen, wo ich die Information in strukturierte Form (z. B. csv, json, aber kein Excel) herbekomme. Falls du was hast, lass es mich gerne wissen

[stoepselinho]

@jlssmt
hab es versucht mit parser aber bekomm es nicht hin

[jlssmt]

@stoepselinho
Was ist denn deine Quelle?

[stoepselinho]

hab mir den Quelltext von corona in zahlen genommen