marcusraitner / COVID-19-Dashboard

Dieses Scriptable-Skript erzeugt ein Widget, das den Verlauf der 7-Tage-Inzidenz und die Auslastung der Intensivbetten am aktuellen Standort anzeigt. Zusätzlich angezeigt wird der aktuelle Wert der 7-Tage-Inzidenz des zugehörigen Bundeslandes und der Impfstatus des Bundeslandes.
GNU General Public License v3.0
210 stars 16 forks source link

Widget-Fehler! #62

Closed HJE0815 closed 3 years ago

HJE0815 commented 3 years ago

Hallo, ich habe auf dem Homebildschirm das Widget 8 mal eingebunden, d.h. für meinen Landkreis und die umliegenden auch. Leider kommt sehr, sehr häufig der Fehler "Error in line 513:48: TypeError: undefined ist not an object (evaluation 'vaccData.data.states')". Ein anderes Mal gehts dann wieder.

Bitte mal prüfen. Danke

HJE0815 commented 3 years ago

Der Fehler kommt auch teilweise in Scriptable bei Einzelausführung auch.

marcusraitner commented 3 years ago

Das Problem ist bekannt. Es liegt an der API hier: https://api.corona-zahlen.org/docs/ Die erlaubt nur 15 Aufrufe pro Minute. Normalerweise kein Problem, aber in deinem Setting kommst du da an die Grenze. Über die API kommen aber nur

Wenn du also bei deinen Widgets teilweise den Impfstatus ausblenden kannst (vac=n) oder den Wert für Deutschland (de=n) oder die berechneten Werte statt der Frozen-Werte verwenden kannst (frozen=n) sollte das Problem verschwinden.

Copperlanding commented 3 years ago

Hab ein ähnliches Problem: „2021-06-07 10:32:58: Error on line 502:48: TypeError: undefined is not an object (evaluating 'vaccData.data.states')“. Auch beim ändern der Werte wie oben vorgeschlagen funktioniert das Widgets leider nicht mehr. Hab Version Version: 1.8.0b. Kann es daran liegen?

marcusraitner commented 3 years ago

Du kannst natürlich mal die neuste Version installieren, aber daran sollte es nicht liegen. Warte einfach mal eine Minute und ruf das Widget nochmal auf; dann sollte es gehen.

Copperlanding commented 3 years ago

Hab die neue Version 19 eingesetzt und die produziert leider den gleichen Fehler: „2021-06-07 10:41:59: Error on line 534:48: TypeError: undefined is not an object (evaluating 'vaccData.data.states')“. Herzliche Grüße

marcusraitner commented 3 years ago

Ich vermute zusätzlich dass es im Moment ein Problem mit der API gibt. Aber mit vac=n;de=n geht es bei mir.

Copperlanding commented 3 years ago

Ja mit vac= n geht’s bei mir jetzt auch. Danke!

Marcus Raitner @.***> schrieb am Mo. 7. Juni 2021 um 10:47:

Ich vermute zusätzlich dass es im Moment ein Problem mit der API gibt. Aber mit vac=n;de=n geht es bei mir.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/marcusraitner/COVID-19-Dashboard/issues/62#issuecomment-855737078, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUISAMWTIJVRBKUFMAHF5CTTRSBS5ANCNFSM46F4JXPA .

marcusraitner commented 3 years ago

Die API liefert einen Fehler: {"error":{"message":"An error occurred.","details":"Cannot read property 'state' of undefined","stack":"TypeError: Cannot read property 'state' of undefined\n at Object.<anonymous> (/usr/src/app/src/data-requests/vaccination.ts:253:15)\n at Generator.next (<anonymous>)\n at fulfilled (/usr/src/app/dist/data-requests/vaccination.js:5:58)\n at runMicrotasks (<anonymous>)\n at processTicksAndRejections (internal/process/task_queues.js:97:5)"}}

Copperlanding commented 3 years ago

Kann ich was ändern?

Marcus Raitner @.***> schrieb am Mo. 7. Juni 2021 um 10:58:

Die API liefert einen Fehler: {"error":{"message":"An error occurred.","details":"Cannot read property 'state' of undefined","stack":"TypeError: Cannot read property 'state' of undefined\n at Object. (/usr/src/app/src/data-requests/vaccination.ts:253:15)\n at Generator.next ()\n at fulfilled (/usr/src/app/dist/data-requests/vaccination.js:5:58)\n at runMicrotasks ()\n at processTicksAndRejections (internal/process/task_queues.js:97:5)"}}

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/marcusraitner/COVID-19-Dashboard/issues/62#issuecomment-855746091, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUISAMWJJBRDBYT6IAECUZDTRSC3XANCNFSM46F4JXPA .

marcusraitner commented 3 years ago

Nein; nur die Parameter entsprechend setzen, damit die Calls der API vermieden werden.

marcusraitner commented 3 years ago

Hier mehr Infos dazu: https://github.com/marlon360/rki-covid-api/issues/233

Kurz: Das RKI hat das Format des Excels geändert. Digitalisierung in D im Jahr 2021 🤦‍♂️

Copperlanding commented 3 years ago

Oh jeh. Deutschland’s RKI in der Steinzeit. Hoffe Ihr großartiges Script funktioniert bald wieder. Ich schau hier jeden Tag mal rein oder besser abonniere Ihren Kanal. Herzliche Grüße und Danke für die tolle Arbeit! Frank Alexander von Keutz

Marcus Raitner @.***> schrieb am Mo. 7. Juni 2021 um 11:51:

Hier mehr Infos dazu: marlon360/rki-covid-api#233 https://github.com/marlon360/rki-covid-api/issues/233

Kurz: Das RKI hat das Format des Excels geändert. Digitalisierung in D im Jahr 2021 🤦‍♂️

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/marcusraitner/COVID-19-Dashboard/issues/62#issuecomment-855784687, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUISAMQ5HZ2UYHZGBZSPBSLTRSJDBANCNFSM46F4JXPA .

marcusraitner commented 3 years ago

Ja, es ist schon erbärmlich … aber bestimmt funktioniert es bald wieder.

kissnofrog commented 3 years ago

Where and how in the script do you enter this vac=n ?

PeterSchimmel commented 3 years ago

In scriptable:

var showVaccination = true;

var showVaccination = false;

Done!

marcusraitner commented 3 years ago

Thanks for your support @PeterSchimmel!

dennerforen commented 3 years ago

In scriptable:

  • Tap the 3 dots on the script's tile, there should be a line

var showVaccination = true;

  • change to

var showVaccination = false;

  • Hit "Done" in the upper left

Done!

https://gist.github.com/marcusraitner/a1b633625d1016498eaaab712461dfc4#gistcomment-3772561

Better way, less errors

marcusraitner commented 3 years ago

You're right, @dennerforen. No need to change the code.

@kissnofrog Just do a long tap on the widget on your home screen; then it will flip and you will have at the bottom a field for the parameters. It is also described here: https://www.iphone-ticker.de/covid-19-dashboard-iphone-widget-zeigt-lokale-inzidenzen-174666/

marcusraitner commented 3 years ago

API has been adjusted to the changed Excel of the RKI; everything's up an running again.