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

Fehler in Zeile 588 #80

Closed ibaga2k closed 3 years ago

ibaga2k commented 3 years ago

Hallo, ein Fehler in Zeile 588:48 wird gemeldet. Dort wird der Impffortschritt ausgewertet. „TypeError: undefined is not an object (evaluating ’vacData.data.states’“ Gruß Karsten

marcusraitner commented 3 years ago

Vermutlich hast du zu oft hintereinander das Widget aufgerufen und / oder mehrere Widgets mit verschiedenen Orten laufen. Die API dahinter (für die Impfung, die Frozen-Werte und den Wert für DE) verträgt nur 15 Aufrufe pro Minute. Probier es bitte nach einer gewissen Zeit nochmal.

Copperlanding commented 3 years ago

Leider nein, ich habe den gleichen Fehler in Zeile 588:48 beim ersten Aufruf des Widgets. Herzliche Grüße

marcusraitner commented 3 years ago

Komisch, weil die Schnittstelle für die Impfungen eigentlich funktioniert; was Probleme macht, ist die Schnittstelle für den Wert für Deutschland bzw. die für die einzelnen Landkreise; vgl. https://github.com/marlon360/rki-covid-api/issues/276

Mit welchen Parametern ruft ihr das Widget auf?

ibaga2k commented 3 years ago

Ich habe folgende Parameter eingegeben: „49.384,11.212;de=y;theme=rki“ mit dem Parameter „de=y“ bekomme ich die Fehlermeldung, ohne diesen Parameter funktioniert es bei mir

Copperlanding commented 3 years ago

const debug = true ;

// Set to true for an image background, false for no image. const imageBackground = false;

// Set to true to reset the widget's background image. const forceImageUpdate = false;

// Show also the incidence for Germany in total var showGermanyValue = true;

// Show also the R-Value (only if showGermanyValue == true) var showRValue = true;

// Toggle showing of vaccination status var showVaccination = true;

// Toggle showing of ICU beds var showIcu = true;

// show daily portion of incidence var showDaily = true;

// Show frozen value for incidence instead of calculating it. var useFrozen = false;

// number of days to show in detail var detail = 5;

// show values for the state var showBl = true;

Bei mir das Gleiche: Läuft mit dem Parameter „de=False“ Herzliche Grüße

ibaga2k @.***> schrieb am Fr. 3. Sept. 2021 um 09:43:

Ich habe folgende Parameter eingegeben: „49.384,11.212;de=y;theme=rki“ mit dem Parameter „de=y“ bekomme ich die Fehlermeldung, ohne diesen Parameter funktioniert 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/80#issuecomment-912330379, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUISAMRQFVTAJFMWL3AWJYLUAB4DPANCNFSM5DK6JDGQ .

marcusraitner commented 3 years ago

Danke euch. Wenn ihr de=n bzw. showGermanyValue = true setzt, sollte es aktuell funktionieren bis die API wieder funktioniert.

Copperlanding commented 3 years ago

Danke, klappt!

Marcus Raitner @.***> schrieb am Fr. 3. Sept. 2021 um 10:00:

Danke euch. Wenn ihr de=n bzw. showGermanyValue = true setzt, sollte es aktuell funktionieren bis die API wieder funktioniert.

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

ibaga2k commented 3 years ago

Bei mir funktioniert nur „de=n“ oder „showGermanyValue = false“, mit „True“ stürzt die Anwendung ab. Aber mit diesem Workaround läuft es. Vielen Dank für die schnelle Hilfe 👍

marcusraitner commented 3 years ago

Ja, stimmt. Mein Fehler oben. Ich meinte natürlich =false

Donigi-77 commented 3 years ago

Sobald ich dies showGermanyValue = true ändere kommt der Fehler 188:279 image Hat sonst jemand noch diesen Fehler?

marcusraitner commented 3 years ago

@Donigi-77 hast du die Koordinaten als Parameter mitgegeben oder werden die vom iPhone ermittelt?

Donigi-77 commented 3 years ago

@marcusraitner wird vom iPhone ermittelt

marcusraitner commented 3 years ago

Ok, dann geht da was schief. Du könntest ja mal in Zeile 416 (also nach dem if else in dem die Location ermittelt wird) diese Ausgabe einbauen console.log(location); und dann das Script gleich im Editor mit dem Play-Symbol dort laufen lassen. Dann solltest du in der Ausgabe (das zweite Icon von links) sehen, was da an Location zurück kommt von deinem iPhone.

Donigi-77 commented 3 years ago

Danke ich werde es versuchen. Nur zur Info wenn ich das Script in der Console ausführe funktioniert es aber nicht wenn ich es über die App mache. Ich gebe Feedback ob das die Lösung ist. Danke nochmals für den schnellen Lösungsvorschlag.

marcusraitner commented 3 years ago

Das ist aber komisch. Was heißt für dich "über die App"? In Scriptable einfach auf das Skript tippen? Mein Vorschlag wird das Problem nicht lösen, ich wollte nur sehen, was in der Location Variable steht … aber wenn es da im Editor funktioniert, wird uns das nicht helfen.

Donigi-77 commented 3 years ago

Entschuldige meine unverständliche Aussage: also wenn ich das Script in der Console ausführe kommt dieses Bild image Und wenn ich dann auf close gehe kommt dies image. Ich hoffe das erklärt meine Aussage 😀

Donigi-77 commented 3 years ago

@marcusraitner So habe deine Änderung eingefügt und meine Parameter in Scriptable gelöscht. Es funktioniert wieder 😄 Danke nochmals

marcusraitner commented 3 years ago

Ja, jetzt verstehe ich das besser. Wie hast du das Skript ausgeführt am Anfang. Warst du da im Editor und dann den Play-Button?

marcusraitner commented 3 years ago

@marcusraitner So habe deine Änderung eingefügt und meine Parameter in Scriptable gelöscht. Es funktioniert wieder 😄 Danke nochmals

Sehr gut. Vermutlich hast du irgendwo bei den Parametern irgendeinen Fehler gehabt. Am besten du holst dir den Code einfach nochmal; dann wird rauch die Beschriftung der Impfungen besser dargestellt.

Donigi-77 commented 3 years ago

@marcusraitner So habe deine Änderung eingefügt und meine Parameter in Scriptable gelöscht. Es funktioniert wieder 😄 Danke nochmals

Sehr gut. Vermutlich hast du irgendwo bei den Parametern irgendeinen Fehler gehabt. Am besten du holst dir den Code einfach nochmal; dann wird rauch die Beschriftung der Impfungen besser dargestellt.

In den Parametern habe ich deine Vorschläge rein wie z.B. rval =y;daily=y; usw. und dein de=n; als des als erstes nicht funktioniert hat. Alles gelöscht und es funktioniert wieder. 👍🏼

marcusraitner commented 3 years ago

Funktioniert wieder.