#erledigt#Wettericons von weather (darkSky) in FTUI darstellen

[hermi]

Ich habe heute in https://darksky.net/dev/docs#data-point-object nachgeschaut. Dort finden sich unter dem icon-property aktuell folgende möglichen Werte:

• clear-day
• clear-night
• rain
• snow
• sleet
• wind
• fog
• cloudy
• partly-cloudy-day
• partly-cloudy-night

Wenn ich das richtig sehe, werden diese im Reading "iconAPI" übertragen.

In widget_weather.js müsste man deshalb ab Zeile 272 folgenden Block einfügen, um die nicht definierten Icon-Definitionen zu ergänzen.:

Code Auswählen Erweitern


        // Dark Sky (IconAPI, see https://darksky.net/dev/docs#data-point-object)
        'clear-day': 'clear.png',
        'clear-night': 'mostly_clear_night.png',
        // 'rain': 'rain.png',      // already defined
        // 'snow': 'snow.png',      // already defined
        // 'sleet': 'sleet.png',    // already defined
        'wind': 'windy.png',
        // 'fog': 'fog.png',        // already defined
        // 'cloudy': 'cloudy.png',  // already defined
        'partly-cloudy-day': 'partly_cloudy.png',
        'partly-cloudy-night': 'partly_cloudy_night.png',


Ich kann einen Pull-Request für https://github.com/knowthelist/fhem-tablet-ui erzeugen. Damit sollte dann der Code offiziell eingepflegt werden können.

[remo]

Hallo ihr Lieben,

wie ist der Stand der Dinge was die Icons betrifft?
Geht es voran?

Der Request scheint noch nicht durch zu sein...

Ich komme auch einfach nicht weiter!
Ich benutze die Icons von Meteocons und das einzige Icon was ich angezeigt bekomme ist "partly_cloudy" ...
Ich habe die "widget_weather.js" hoch- und runter-editiert - kein zufriedenstellendes Ergebnis.
Hat vielleicht noch jemand einen Tipp für mich?


-remo

[hermi]

Hallo remo,

ich wollte zuerst noch auf Feedback von Helmi55 und Tommy82 warten, ob meine Änderungen mit ihren übereinstimmen.

Und dass eventuell setstate diese Änderung hier mitbekommt und sie auf dem kleinen Dienstweg einpflegt (liest er alle Nachrichten in diesem Board mit?).

Die Änderung ansich ist nicht kompliziert:
Im aktuellen widget_weather.js ist der o.g. Code im Bereich

Code Auswählen Erweitern

var kleinklimamap = { zwischen den Blöcken "Weather (YAHOO) en" und "PROPLANTA" einzufügen uns speichern. Bei mir ist das ab Zeile 272 (siehe Anhang).
Danach die TabletUI-Oberfläche neu laden.

[Tommy82]

Hallo remo,

ich wollte zuerst noch auf Feedback von Helmi55 und Tommy82 warten, ob meine Änderungen mit ihren übereinstimmen.

Und dass eventuell setstate diese Änderung hier mitbekommt und sie auf dem kleinen Dienstweg einpflegt (liest er alle Nachrichten in diesem Board mit?).

Hi,
aus meiner Sicht passt das so und wäre schön wenn es eingecheckt würde.

[remo]

Guten Morgen zusammen,

vielen Dank für eure Rückmeldungen.
Ich Nutze zwar nicht das kleinklima-ImageSet,
aber diese Variante der neuen Definitionen habe ich schon getestet.
Ich nutze Meteocons.

Die Logik ist mir schon klar,
aber eigenartigerweise hat das nicht funktioniert.
Muss unter der Sektion translationmap auch noch etwas ergänzt werden?

Gruß

[hermi]

Ok, verstehe...

Dann schau mal, ob es klappt, wenn Du diese Zeilen ab Zeile 102 einfügst:

Code Auswählen Erweitern


        // Dark Sky (IconAPI, see https://darksky.net/dev/docs#data-point-object)
        'clear-day': 'B',
        'clear-night': '2',
        // 'rain': 'R',      // already defined
        // 'snow': 'W',      // already defined
        // 'sleet': 'X',     // already defined
        'wind': 'S',
        // 'fog': 'F',       // already defined
        // 'cloudy': '5',    // already defined
        'partly-cloudy-day': 'H',
        'partly-cloudy-night': '5',


Die Zuordnung der Icons zu den Zahlen/Buchstaben habe ich bei einer Schnellsuche nicht gefunden. Ich habe versucht die Zuordnung so ähnlich wie bei Yahoo, Proplanta usw. zu machen.


Ob man der Vollständigkeit halber auch noch die weathericonsmap anpassen müsste, kann ich nicht sagen. So gut verstehe ich den Code auch wieder nicht.

[Helmi55]

Hallo
und sorry fürs lange nicht melden. War Busy.
Ich habe die fehlenden Ausdrücke einfach ab 272 eingefügt. Heute ist mir aufgefallen dass 'Fog' bei mir fehlte.
Jetzt passt wieder
Gruß
Helmut

[hermi]

Gut, dann werde ich noch auf die Rückmeldung von Remo warten. Wenn bei ihm die meteocons-Änderung funktioniert, werde ich eine Pull-Request erstellen.

Wann dieser dann in den Hauptzweig eingepflegt wird, kann ich nicht sagen.

[remo]

Danke für eure Bemühungen,
das sieht erstmal alles ganz gut aus.

Request kann starten

[hermi]

Ok, erledigt: https://github.com/knowthelist/fhem-tablet-ui/pull/237

Jetzt hängt es an setstate, wann die Änderung in den master-Zweig eingepflegt wird.

Schönes Wochenende!
Hermann

[remo]

Na, wie sieht's aus?
Gibt es schon Fortschritte?

[somansch]

Ich arbeite gerade an einer Komplettüberarbeitung des Weather_Widgets. Näheres später in einem neuen Threat 

VG
Andreas

[remo]

@somansch:
klingt super - bin gespannt

Wirst du dann den Link zum neuen Thread hier posten?

[somansch]

@somansch:
klingt super - bin gespannt

Wirst du dann den Link zum neuen Thread hier posten?

Ich habe eine neue Version des "Weather_Widgets" zur Darstellung der Icons in FTUI zum Testen bereitgestellt. Diese Version unterstützt DarkSky, OpenWeather, ProPlanta und DWD https://forum.fhem.de/index.php/topic,96954.0.html

Viele Grüße
Andreas

[myhome]

Heute nach Update keine Icons mehr. Nutze Proplanta .  In der CSS von wdtimer ist auch was krumm, die Version aus Form geht dort zumindest