UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[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

[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

[jlssmt]

@stoepselinho
ja das ist halt instabil.
wie ich das sehe wird die inzidenz nur von deutschland getrackt. es ist schon schwer die 7-tage inzidenzen für bspw. UK zu googlen.
Alles was ich so finde, beruht auf eigenen Berechnungen der Entwickler.

Die Tabelle hier zu scrapen ist auch schwierig, da sie mehrere Seiten hat und die Daten asynchron lädt. Da brauchts dann mWn. wieder puppeteer usw... wenns überhaupt irgendwie stabil möglich ist.
https://www.corona-in-zahlen.de/weltweit/
man könnte den entwickler anfragen, ob er eine api bereitstellt. aber ich weiß nicht ob sein server dem load gewappnet ist.

[jlssmt]

jhu liefert die inzidenz inkl. historie. damit könnte man die 7-tage-inzidenz einfach selbst berechnen.
leider für deutschland, spanien und andere nur pro bundesland und nicht gesamt.
bei manchen ländern würde uns das helfen, aber nicht bei den wichtigen.

https://github.com/CSSEGISandData/COVID-19/tree/master/csse_covid_19_data/csse_covid_19_daily_reports

[Dutchman]

0.8.0-1 released, Umstellung von RKI data per Excel auf eine validierte API hoffentlich damit auch endlich diese werte unter Kontrolle :)

Bitte teste, ich moechte die version gerne nach stable bringen wegen den Impfdaten und Fehler beim JS-ccontroller 3.x

[sigi234]

@dutchman

coronavirus-statistics.0
2021-08-11 12:03:17.170	warn	Cannot handle vaccination data for Hungary, if this error continues please report a bug to the developer! Totals: {"date":"2021-08-10","people_vaccinated":5658623,"people_fully_vaccinated":5497599,"people_vaccinated_per_hundred":58.58,"people_fully_vaccinated_per_hundred":56.91}

coronavirus-statistics.0
2021-08-11 12:03:00.026	warn	Got terminate signal. Checking desired PID: 0 vs own PID 13532

coronavirus-statistics.0
2021-08-11 12:02:57.295	warn	Cannot handle vaccination data for China, if this error continues please report a bug to the developer! Totals: {"date":"2021-08-10","total_vaccinations":1808092000,"daily_vaccinations_raw":25567000,"daily_vaccinations":14248000,"total_vaccinations_per_hundred":125.62,"daily_vaccinations_per_million":9899}

[jlssmt]

@Dutchman

Das liegt daran, dass unsere Problemländer China und Ungarn die Daten unvollständig/ anders bereitstellen.
Ungarn hat keine "total_vaccinations" und China splittet nicht auf Erst- und Zeitimpfung auf:

[
  {
    "country": "China",
    "iso_code": "CHN",
    "data": [
      {
        "date": "2021-08-10",
        "total_vaccinations": 1808092000,
        "daily_vaccinations_raw": 25567000,
        "daily_vaccinations": 14248000,
        "total_vaccinations_per_hundred": 125.62,
        "daily_vaccinations_per_million": 9899
      }
    ]
  },
  {
    "country": "Hungary",
    "iso_code": "HUN",
    "data": [
      {
        "date": "2021-08-10",
        "people_vaccinated": 5658623,
        "people_fully_vaccinated": 5497599,
        "people_vaccinated_per_hundred": 58.58,
        "people_fully_vaccinated_per_hundred": 56.91
      }
    ]
  }
]

Muss hier angepasst werden:

	async writeVaccinationDataForCountry(country, data) {
		if (data
			&& data.people_vaccinated
			&& data.people_fully_vaccinated
			&& data.total_vaccinations
			&& data.people_vaccinated_per_hundred
			&& data.people_fully_vaccinated_per_hundred) {

			[...]
		} else {
			this.log.warn(`Cannot handle vaccination data for ${country}, if this error continues please report a bug to the developer! Totals: ${JSON.stringify(data)}`);
		}
	}