rphl / corona-widget

COVID-19 Widget ( Inzidenz / Impfquoten / Hospitalisierungen ) für iOS innerhalb Deutschlands 🇩🇪 (Kreis/Stadt + Bundesland + Trend)
Other
648 stars 95 forks source link

⁉️ Fragen, Feedback, etc. #6

Open rphl opened 3 years ago

rphl commented 3 years ago
WillyWallace commented 3 years ago

Hallo rphl, tolle Arbeit, die du hier leistest. Wirklich prima. Ich frage mich, ob du irgendwelche Entwickler-Tools nutzt, bei denen man links den Code und rechts die Ausgabe sieht. Wie ist da deine Arbeitsweise.

Dann habe ich noch eine Frage. Bei mir passt der Inhalt ständig nicht komplett in die Medium-Widgets. Die sind immer abgeschnitten. Woran liegt das?

IMG_50C8710DC473-1

Beste Grüße, Andi

rphl commented 3 years ago

tolle Arbeit, die du hier leistest. Wirklich prima. Ich frage mich, ob du irgendwelche Entwickler-Tools nutzt, bei denen man links den Code und rechts die Ausgabe sieht. Wie ist da deine Arbeitsweise.

Danke. Arbeite am Mac+VSCode und synce die Änderungen instant via iCloud auf die Geräte. Da sehe ich direkt die Änderungen / Logs. Muss aber noch schauen wie sich das evtl. optimieren lässt.

Das alte Widget hat leider noch viele feste breiten. Dies ändert sich im Update, da wird es deutlich flexibler. Hast du auch evtl. den Anzeigezoom an? Was für ein Gerät ist es?

WillyWallace commented 3 years ago

Ah, danke für die Antwort.

Ich habe das iPhone Se (1st Gen.) Wo finde ich denn den Anzeigezoom?

rphl commented 3 years ago

@WillyWallace Müsste Anzeige & Helligkeit > Anzeigezoom sein

WillyWallace commented 3 years ago

@WillyWallace Müsste Anzeige & Helligkeit > Anzeigezoom sein

Dachte ich auch, aber da ist nichts. Hab iOS 14.1

tomacco81 commented 3 years ago

Das gibt‘s glaub nur bei Geräten mit größerem Display... Das iPhone 5/5S/SE hat ja nur ein 4“ Display - evtl liegt‘s daran 🤔

rphl commented 3 years ago

Habe mal ein Update online gestellt. Gerne testen. Das ist jetzt etwas flexibler.

STJIKA commented 3 years ago

96455B4A-8D8E-4F84-B94F-90417810DD56 ...wäre es eine Option das widget ein wenig umzustellen und so den Inzidenzwert für Deutschland noch unterzubringen....????

STJIKA commented 3 years ago

E0CFC3B1-D703-49A4-A2E9-E9923BA1FD59 ...noch eine optische Kleinigkeit ist mir aufgefallen...denke es sähe noch etwas perfekter aus wenn der „Einzug“/Ausrichtung siehe Pic „gleich“ wäre....??? (iPad Pro 12,9 Zoll (2018))

rphl commented 3 years ago

Den Einzug werde ich beim nächsten mal anpassen, aber Wahrscheinlich nur den Kasten ausrichten. Denke da es so Stimmiger ist. Werde aber auch mal etwas rumprobieren ;-)

tooomm commented 3 years ago

Wie hier im Medium Widget (linke Angabe) zu sehen sind die beiden ineinanderliegenden Box scheinbar nicht immer korrekt ausgerichtet.

dennerforen commented 3 years ago

Mal eine praktische Frage, ich bin ja bei GitHub und ghist hub ein Neuling und habe die nur, um die tollen Widgets zu nutzen.

Hier bei GitHub in der iOS App ist das kopieren von Code relativ umständlich, ich kann nur die 40 sichtbaren Zeilen markieren, dann warten bis das iPad das in die Zwischenablage kopiert hat und dann scrollen, das die nächsten 40 Zeilen weiter von mir markiert werden können, bei ghisthub hat die #eite weitergescrollt, wenn ich mit dem markieren unten am Boden war, aber hier in der App geht das nicht.

Gibt es einen cleveren Weg, den Code zu kopieren?

In dem Betten Script, hat der Autor das elegant gelöst, er hat einen Zweizeiler geschrieben, der das Haupt Script aufruft. Dort brauche ich also nicht copy und Paste machen, sondern habe immer den aktuellsten Code im Script.

rphl commented 3 years ago

Wegen der automatischen Installation hatte ich mal meine Idee mit der "Kurzbefehle" gepostet: Zum Problem bei Punkt 4. hatte jemand schon eine Lösung gespostet, weiss aber nicht mehr wo 🤷‍♂️ Original post Noch mal hier reinkopiert:

Idee wie mal Scriptable "fast" automatisch updaten könnte: Kurzbefehle Aktion anlegen:

  1. RAW URL kopieren
  2. Neuen Shortcut in Kurzbefehle App anlegen IMG_5442

4. Hier müssten man noch rausfinden wie die Datei überschrieben werden kann. Aktuell erstellt er einen neuen Eintrag Wenn also einer Zeit hat das mal zu checken wäre super

tooomm commented 3 years ago

Versuch es einfach mal über den Safari Browser direkt, ohne App. Eventuell noch auf Raw oben klicken...

rphl commented 3 years ago

Wie hier im Medium Widget (linke Angabe) zu sehen sind die beiden ineinanderliegenden Box scheinbar nicht immer korrekt ausgerichtet.

@tooomm Das müsste aber schon längst gefixed sein 🤔

firebowl commented 3 years ago

Ich finde die Idee von @dennerforen irgendwie recht praktisch. 😉 Mich nervt es auch ein wenig, nich den ganzen Quellcode kopieren zu können. Ich benutze am iPhone entweder die GitHub App oder Microsoft Edge und in beiden gibt beim markieren die Option "Alles" nicht. Hier ist das Intensivbettenwidget, das den eigentlichen Code erst beim Aufruf oder aktualisieren lädt: https://github.com/Keyes/its-beds-widget Vielleicht wäre das ja auch ne Option.

JoeGit42 commented 3 years ago

Ich finde die Idee von @dennerforen irgendwie recht praktisch. 😉 Mich nervt es auch ein wenig, nich den ganzen Quellcode kopieren zu können. Ich benutze am iPhone entweder die GitHub App oder Microsoft Edge und in beiden gibt beim markieren die Option "Alles" nicht. Hier ist das Intensivbettenwidget, das den eigentlichen Code erst beim Aufruf oder aktualisieren lädt: https://github.com/Keyes/its-beds-widget

Vielleicht wäre das ja auch ne Option.

Beim Nachladen sehe ich spontan zwei Probleme:

  1. Nutzer, die den Code anpassen möchten haben es schwerer
  2. Code nachzuladen öffnet eine gewissen Sicherheitslücke

Ansonsten ist das natürlich eine coole Möglichkeit.

firebowl commented 3 years ago

Ich finde die Idee von @dennerforen irgendwie recht praktisch. 😉

Mich nervt es auch ein wenig, nich den ganzen Quellcode kopieren zu können. Ich benutze am iPhone entweder die GitHub App oder Microsoft Edge und in beiden gibt beim markieren die Option "Alles" nicht.

Hier ist das Intensivbettenwidget, das den eigentlichen Code erst beim Aufruf oder aktualisieren lädt:

https://github.com/Keyes/its-beds-widget

Vielleicht wäre das ja auch ne Option.

Beim Nachladen sehe ich spontan zwei Probleme:

  1. Nutzer, die den Code anpassen möchten haben es schwerer

  2. Code nachzuladen öffnet eine gewissen Sicherheitslücke

Ansonsten ist das natürlich eine coole Möglichkeit.

Bei beiden Punkten bin ich absolut bei dir. Keyes lässt den Leuten aber die Wahl. Die faulen können den Zweizeiler nehmen und wer will kann den original Code verwenden. Beim Nachladen hast Du natürlich vollkommen recht aber ich behaupte mal, 90% prüfen auch jetzt nicht was der Code so treibt, den sie per copy & paste benutzen. ;) Ich kann mit beiden Varianten leben. 😃

dennerforen commented 3 years ago

Versuch es einfach mal über den Safari Browser direkt,

Danke, auch ne Idee. Gebe Feedback

Nachtrag: Sieht in Safari auch wieder anders aus, aber ich habe den Code gefunden und dort geht es mit scrollmarkierung. Danke für den Tipp

RosoV89 commented 3 years ago

Dass du das jetzt hier umgestellt hast, ist an mir vorbeigegangen, daher habe ich die letzten 4-5 Tage die Updates nicht mitbekommen.

Eine Frage habe ich: Ich verstehe die Balkendarstellung noch nicht wirklich. Wachsen die Balken jetzt quasi nach unten? oder was soll die Darstellung verdeutlichen?

rphl commented 3 years ago

Die Diagramme zeigen weiterhin die Inzidenz der letzten 7 Tage. Es ist aber ein Update in Vorbereitung das im Diagramm die reinen neuen Fälle der letzten 7Tage anzeigen soll. Siehe hier: #22

Wachsen die Balken jetzt quasi nach unten? oder was soll die Darstellung verdeutlichen?

Ist ein reine optische Entscheidung. Die Zahlen sind also nicht negativ. Der Trend entspricht also weiterhin den Inzidenz-Daten (Siehe aber auch #22)

RosoV89 commented 3 years ago

Ok verstehe. Jetzt bin ich aber wieder up to Date und verfolge die Entwicklung jetzt weiterhin

kevinkub commented 3 years ago

Ich finde die Idee von @dennerforen irgendwie recht praktisch. 😉 Mich nervt es auch ein wenig, nich den ganzen Quellcode kopieren zu können. Ich benutze am iPhone entweder die GitHub App oder Microsoft Edge und in beiden gibt beim markieren die Option "Alles" nicht.

Alternativ könntest du https://scriptdu.de nutzen. Das ist eine Art Paket-Manager, den ich für Scriptable gebaut habe. Der Charm dabei: Sämtliche Logik bleibt in dem Scriptable-Script selbst. Du kannst manuell updaten - musst aber nicht. Und es funktioniert mit jedem bestehenden Widget.

Download with ScriptDude

firebowl commented 3 years ago

@kevinkub Saucool, zumindest das installieren ist so ein Traum. Wie läuft das aktualisieren ab? Einfach das ScriptDude Script aufrufen?

dennerforen commented 3 years ago

Danke Kevin, habe es mal getestet, sieht vielversprechend aus.

kevinkub commented 3 years ago

@kevinkub Saucool, zumindest das installieren ist so ein Traum. Wie läuft das aktualisieren ab? Einfach das ScriptDude Script aufrufen?

Exakt. Script starten und wenn es Updates gibt erscheinen die oben.

RosoV89 commented 3 years ago

Das Skript kann seit heute Morgen nicht mehr geladen werden. Noch jemand das Problem?

dennerforen commented 3 years ago

IMG_0755

Liegt an der Datenquelle

RosoV89 commented 3 years ago

Hat wohl noch niemand bemerkt, weil in den Nachrichten der Fokus auf die US Wahl liegt... von Corona habe ich heute bisher nichts gehört 🙈

rphl commented 3 years ago

Ja leider -.- Unabhängig von den nicht aktuellen Daten, scheint es generell Probleme mit der Api zu geben (Gegeben zu haben) Siehe auch hier: https://github.com/rphl/corona-widget/issues/28

moritzdietz commented 3 years ago

Mich hätte interessiert welche Elemente in dem Widget welche Werte Wiederspiegeln. Kann dann auch ein PR stellen um die README anzupassen. Habe versucht mir aus dem Code das Ganze selbst zu beantworten, aber bin nicht schlauer geworden. Im gist stand auch nichts.

rphl commented 3 years ago

Heute Abend/Morgen Vormittag gibt es ein Update zum Widget. Versuche dann direkt die Readme Upzudaten :-)

tooomm commented 3 years ago

@moritzdietz siehe https://github.com/rphl/corona-widget/issues/23 und ganz besonders https://github.com/rphl/corona-widget/issues/22#issuecomment-720464265

rphl commented 3 years ago

So Update ist online. Was es neues gibt steht hier: README und Changelog

IMG_5438

Gerne testen (JSON backup zuvor erstellen :-)). Und Feedback geben

Konnte die Version ohne iCloud nicht ausgiebig testen. Gerne auch hier Feedback an mich


Mich hätte interessiert welche Elemente in dem Widget welche Werte Wiederspiegeln. Kann dann auch ein PR stellen um die README anzupassen. Habe versucht mir aus dem Code das Ganze selbst zu beantworten, aber bin nicht schlauer geworden. Im gist stand auch nichts.

@moritzdietz Habe die readme aktualisiert.


@moritzdietz siehe #23 und ganz besonders #22 (comment)

@tooomm Ist gefixed :-)

TheSpirit commented 3 years ago

Wollte gerne 28 Tage im Diagramm anzeigen lassen. Das klappt aber leider nicht. Meinst du, du bekommst das hin? Danke für die super Arbeit

RosoV89 commented 3 years ago

5FAC072C-C0DD-4F8D-8F35-1D5F48D1CB60

Sieht super aus.

Kannst du die Zahl oben rechts noch bündig machen, siehe Linie.

Und unten den Abstand etwas verringern, damit die Graphen auch gut lesbar sind, wenn hier 21 Tage oder mehr angezeigt werden.

rphl commented 3 years ago

Wollte gerne 28 Tage im Diagramm anzeigen lassen. Das klappt aber leider nicht. Meinst du, du bekommst das hin? Danke für die super Arbeit

@TheSpirit Im Script die Config für Caching/Diagram auf 28 stellen und warten. Dann werden im Diagramm max 28 Tage dargestellt

Ob das platztechnisch dann noch gut aussieht steht, sei mal dahingestellt. Denke 14 tage ist schon ein guter Wert, der noch schön darzustellen ist.

TheSpirit commented 3 years ago

Hab beides auf 28 gestellt, dann wird aber gar kein Diagramm mehr angezeigt. Platz wäre ja aber noch denke ich

dennerforen commented 3 years ago

@rphl sieht top aus. Danke für das Update.

@TheSpirit ich denke nicht, das das Sinn macht. Die covid App zeigt die letzten 14 Tage an, weil du dann durch bist. Wenn der Trend der letzten 14 Tage sichtbar ist, ändert sich ja nicht viel.

simonCGN commented 3 years ago

Oh, Du hast die Erläuterung mit einem Screenshot gemacht. Noch viel besser als ich es mir vorgestellt hatte. Sehr hilfreich. Herzlichen Dank!

TheSpirit commented 3 years ago

@TheSpirit ich denke nicht, das das Sinn macht. Die covid App zeigt die letzten 14 Tage an, weil du dann durch bist. Wenn der Trend der letzten 14 Tage sichtbar ist, ändert sich ja nicht viel.

stimmt, daran hatte ich garnicht gedacht

TheSpirit commented 3 years ago

Würde es nicht Sinn machen den R-Wert auch immer mit zwei Nachkommastellen anzugeben?

rphl commented 3 years ago

Ah sehe ich gerade auch, zeigt nur eine 1 an. Da hat das RKI sich wohl die Nachkommostellen gesparrt.

TheSpirit commented 3 years ago

Du könntest ja trotzdem immer zwei anzeigen.

youmakemyday commented 3 years ago

bei ner älteren Version (glaub vom 01.11.) von deinem Widget wird noch 1,02 angezeigt ... nur bei der aktuellen Version wird auf 1 gerundet (?)

Dann sollte es ja eigentlich nicht an den Daten vom RKI liegen oder wird der Faktor jetzt woanders hergeholt?

rphl commented 3 years ago

1,02 ist aus Richtig 🤔 habe mal ein update online gestellt. Das wieder auf die Feldnamen in der RKI CSV geht. Hier scheint es sich doch mal öfter zu verschieben 🤷‍♂️

youmakemyday commented 3 years ago

hat geklappt, danke dir.

firebowl commented 3 years ago

Vielleicht hab ich das die letzten Tage verschlafen, deshalb frag ich einfach mal. Warum benutzen wir nen eigenen geschätzten R-Wert und nicht den offiziellen vom RKI? Offiziell ist heute ca. 0,8 und wir nehmen 1,02.

Eeewald85 commented 3 years ago

Warum benutzen wir nen eigenen geschätzten R-Wert und nicht den offiziellen vom RKI? Offiziell ist heute ca. 0,8 und wir nehmen 1,02.

Selbst geschätzt wird der R-Wert nicht, sondern tatsächlich der Wert vom RKI genommen. Aber das RKI berechnet da selbst zwei verschiedene Werte: Die ca. 0,8 scheint der R-Wert in Bezug auf die letzten 4 Tage (?) zu sein, das Script benutzt aber den R-Wert der letzten 7 Tage. Dieser Wert fehlt für heute aber in der CSV Datei vom RKI noch, deswegen wird sicherlich noch der Wert von gestern angezeigt

Hier ist die CSV Datei, die ausgelesen wird: https://www.rki.de/DE/Content/InfAZ/N/Neuartiges_Coronavirus/Projekte_RKI/Nowcasting_Zahlen_csv.csv?__blob=publicationFile

UweLoyal commented 3 years ago

So Update ist online. Was es neues gibt steht hier: README und Changelog

IMG_5438

Gerne testen (JSON backup zuvor erstellen :-)). Und Feedback geben

Konnte die Version ohne iCloud nicht ausgiebig testen. Gerne auch hier Feedback an mich

Mich hätte interessiert welche Elemente in dem Widget welche Werte Wiederspiegeln. Kann dann auch ein PR stellen um die README anzupassen. Habe versucht mir aus dem Code das Ganze selbst zu beantworten, aber bin nicht schlauer geworden. Im gist stand auch nichts.

@moritzdietz Habe die readme aktualisiert.

@moritzdietz siehe #23 und ganz besonders #22 (comment)

@tooomm Ist gefixed :-)

Ist jetzt sehr gut geworden! Auch das Bild mit der Inhaltsbeschreibung erklärt jetzt alles. Vorschlag: Antippen des Widget öffnet Screen mit Erklärung.

Weiter so!

LG Uwe Loyal