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

[jlssmt]

@dutchman said in Test Coronavirus Statistics for ioBroker:

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

ich kann das gerne übernehmen, aber nur, wenn wir die arbeit nicht doppelt machen

da waehre ich dir sehr dankbar, ich baue gerade am WLED und habe fuer corona heute die RKI daten umgestellt auf API wollte den corona adapter eigentlich in ruhe lassen

PR ist gestellt
habs für china und ungarn cross gecheckt

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@dutchman said in Test Coronavirus Statistics for ioBroker:

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

ich kann das gerne übernehmen, aber nur, wenn wir die arbeit nicht doppelt machen

da waehre ich dir sehr dankbar, ich baue gerade am WLED und habe fuer corona heute die RKI daten umgestellt auf API wollte den corona adapter eigentlich in ruhe lassen

PR ist gestellt
habs für china und ungarn cross gecheckt

danke ! ich füge dan noch die neue state Definitionen hinzu und mache ne neue release:

	2021-08-11 14:23:33.078	warn	State attribute definition missing for daily_vaccinations_per_million

coronavirus-statistics.0
2021-08-11 14:23:33.073	warn	State attribute definition missing for people_fully_vaccinated_per_hundred

coronavirus-statistics.0
2021-08-11 14:23:33.068	warn	State attribute definition missing for people_vaccinated_per_hundred

coronavirus-statistics.0
2021-08-11 14:23:33.062	warn	State attribute definition missing for total_vaccinations_per_hundred

coronavirus-statistics.0
2021-08-11 14:23:33.057	warn	State attribute definition missing for daily_vaccinations

coronavirus-statistics.0
2021-08-11 14:23:33.052	warn	State attribute definition missing for daily_vaccinations_raw

coronavirus-statistics.0
2021-08-11 14:23:33.047	warn	State attribute definition missing for people_fully_vaccinated

coronavirus-statistics.0
2021-08-11 14:23:33.042	warn	State attribute definition missing for people_vaccinated

coronavirus-statistics.0
2021-08-11 14:23:33.035	warn	State attribute definition missing for total_vaccinations

coronavirus-statistics.0
2021-08-11 14:23:33.028	warn	State attribute definition missing for date

[Dutchman]

@jlssmt

hmmm.... da passt aber was nicht

ich glaube das wird versacht da nicht alle Länder daten haben also kan der Fehler auch von error nach debug ?

[jlssmt]

@dutchman
jo grad nochmal nachgeschaut, was BLM ist. irgendwas französisches. und halt nicht in den werten der API.
weiß nicht ob error oder debug. was mich mehr nervt ist, dass die message so oft kommt.
ich schau dann nochmal rein, woran das liegt.

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@dutchman
jo grad nochmal nachgeschaut, was BLM ist. irgendwas französisches. und halt nicht in den werten der API.
weiß nicht ob error oder debug. was mich mehr nervt ist, dass die message so oft kommt.
ich schau dann nochmal rein, woran das liegt.

cool thanks dan warte ich darauf

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

weiß nicht ob error oder debug. was mich mehr nervt ist, dass die message so oft kommt.

@jlssmt warte bitte habs schon gefunden !

[Dutchman]

@jlssmt problem gefunden, die Funktion ware eine loop zu tief und wurde fuer jeden state des landes aufgerufen anstatt einmal darum auch soviel log eintrage.

Habe jetzt :

• Den aufruf 1 niveau nach oben gezogen
• Error handling/warning optimalisiert

https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/commit/0056e4041f98a5b59c42094eff3b1498ac024342

Code auf git, magst mal drüber schauen bitte ob es so passt ? Wen ok nur noch die neue Attribute Definitionen und dan koennen wir wieder releasen

ps, wen du eine await Funktion aufrufst immer n try/catch drum herum sonst knallt es when der Promis lehr bleibt.
Im catch can man dan den feeler abfangen (siehe auch commit auf git wo ich das ergänzt habe)

[jlssmt]

@dutchman

sieht gut aus, also ist mmn. jetzt an der richtigen stelle.

danke für die erinnerung bei await. benutze ich zu selten, da ich das meiste über promise chains mache und dort nen catch block hab.

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

danke für die erinnerung bei await. benutze ich zu selten, da ich das meiste über promise chains mache und dort nen catch block hab.

jup da macht es sin es weck zu lassen, danke fuer deine Hilfe und Bemühungen !

[Dutchman]

0.8.0-2 released (auf git, ab morgen im admin!) happy testing

0.8.0-2 (2021-08-11)

(jlssmt) Added vaccination data for all countries
(DutchmanNL) added definitions for vaccination states
(DutchmanNL) Changeover to validated API for German vaccination data of Bundesländer

[varg]

@dutchman

habe jetzt die ganze welt (vaccination) in den Objekten. Soll das so sein ? Ausgewählt habe ich nur Germany.
Löschen der Objekte bewirkt hier nichts. Wird alles gleich neu angelegt.

Ich mein ja nur. Von Afghanistan bis Zimbabwe muss ich nicht zwingend haben

[Feuersturm]

@Dutchman @jlssmt Kann ich bestätigen, es werden alle Lände aufgelistet in den Objekten obwohl diese nicht konfiguriert sind. Bei jedem Land das nicht konfiguriert ist taucht aber nur der Ordner coronavirus-statistics.0.*.Vaccination mit seinen Datenpunkten auf.

https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/issues/306

[jlssmt]

@feuersturm @varg

das if fehlt. entweder das braucht es zusätzlich oben nochmal, oder es kann aus der for-schleife raus und um alles rum. falls ich lust hab, denk ich mich da morgen mal rein, oder @Dutchman weiß es schon direkt

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@feuersturm @varg

das if fehlt. entweder das braucht es zusätzlich oben nochmal, oder es kann aus der for-schleife raus und um alles rum. falls ich lust hab, denk ich mich da morgen mal rein, oder @Dutchman weiß es schon direkt

Schauen wir uns morgen an incl cleanup

[jlssmt]

@dutchman

ich hab jetzt nen kleinen PR Hotfix ohne neue Version gestellt.
Als ich "damals" die Vaccination umgebaut habe, hab ich den richtigen Ort gesucht und dann die if gefunden, daher ist es dort in der For Schleife gelandet. Und beim Test ists mir nicht aufgefallen, weil es für meine Länder halt ging
Ich hab geschaut ob man es irgendwie optimieren könnte, aber dann müsste man die ganze Funktion anfassen und das ist mir aktuell zu viel Aufwand mich da rein zu denken. Außerdem weiß ich gar nicht ob du das willst, du musst dich bei der Kontrolle auch reindenken und es gibt dann evtl wieder Bugs...

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

@dutchman

ich hab jetzt nen kleinen PR Hotfix ohne neue Version gestellt.
Als ich "damals" die Vaccination umgebaut habe, hab ich den richtigen Ort gesucht und dann die if gefunden, daher ist es dort in der For Schleife gelandet. Und beim Test ists mir nicht aufgefallen, weil es für meine Länder halt ging
Ich hab geschaut ob man es irgendwie optimieren könnte, aber dann müsste man die ganze Funktion anfassen und das ist mir aktuell zu viel Aufwand mich da rein zu denken. Außerdem weiß ich gar nicht ob du das willst, du musst dich bei der Kontrolle auch reindenken und es gibt dann evtl wieder Bugs...

thanks gesehen, wir hatten den selben Gedanken, optimiere gerade noch n bissl damit er auch alles wieder aufräumt

[jlssmt]

@dutchman
Zum Aufräumen ist mir auch was aufgefallen:
Ich finde die häufigen localDeleteStates() machen das ganze sehr unübersichtlich. Kann man nicht einfach den ganzen übergeordneten State löschen?

Also das hier:

						await this.localDeleteState(`${channelName}._Impfungen.rkiImpfungenProTausend`);
						await this.localDeleteState(`${channelName}._Impfungen.rkiDifferenzVortag`);
						await this.localDeleteState(`${channelName}._Impfungen.rkiIndikationAlter`);
						await this.localDeleteState(`${channelName}._Impfungen.rkiIndikationBeruf`);
						await this.localDeleteState(`${channelName}._Impfungen.rkiIndikationMedizinisch`);
						await this.localDeleteState(`${channelName}._Impfungen.rkiImpfungePflegeheim`);

zu dem hier:

						await this.localDeleteState(`${channelName}._Impfungen`);

So hab ichs bei Vaccination gemacht und das könnte man im Projekt an vielen Stellen so machen.

[Dutchman]

@jlssmt sagte in Test Coronavirus Statistics for ioBroker:

Ich finde die häufigen localDeleteStates() machen das ganze sehr unübersichtlich. Kann man nicht einfach den ganzen übergeordneten State löschen?

genau das baue ich gerade um, momentan werden nur ordner erstellt in der neuen version sind das schön devices und channel Strukturen und kan ich das komplette land mi einen Befehl aufräumen.
Der localDelete geht dafuer nicht der er nur states löscht keinen verschachtelten order.

Habs fast fertig pushe gleich auf git

[sigi234]

@dutchman

Also braucht der User selbst nichts machen?

[Dutchman]

@sigi234 sagte in Test Coronavirus Statistics for ioBroker:

@dutchman

Also braucht der User selbst nichts machen?

nope, die næste version räumt meinen Fehler wieder auf