Yahoo Wetter

Beta-User · 03 September 2020, 13:25:55

Die aktuelle MAINTAINER.txt gibt das so wieder, und es ist auch folgerichtig, siehe https://forum.fhem.de/index.php/topic,113722.msg1079869.html#msg1079869.

(Es wundert mich ein bißchen, dass Christoph das nicht schon länger gefixt hat; das Problem ist ja nicht neu... Rudi hat aber bestimmt auch nichts dagegen, wenn du ihm einen Patch lieferst, der auf was passendes aus einem Weather-Device zugreift

. Aus älteren Diskussionen meine ich aber mitgenommen zu haben, dass es dafür keinen wirklichen Ersatz gibt?).

yersinia · 03 September 2020, 13:33:55

Aber es gibt doch schon ein Thread für Twilight und dem Thema Yahoo Weather -> 59_Twilight.pm - Funktioniert seit dem 3.1.19 nicht mehr - Yahoo API Umstellung
Gefühlt ist die Twilight Modul Entwicklung erstmal pausiert.

CoolTux · 03 September 2020, 13:38:28

Zitat von: Beta-User am 03 September 2020, 13:25:55
Die aktuelle MAINTAINER.txt gibt das so wieder, und es ist auch folgerichtig, siehe https://forum.fhem.de/index.php/topic,113722.msg1079869.html#msg1079869.

(Es wundert mich ein bißchen, dass Christoph das nicht schon länger gefixt hat; das Problem ist ja nicht neu... Rudi hat aber bestimmt auch nichts dagegen, wenn du ihm einen Patch lieferst, der auf was passendes aus einem Weather-Device zugreift . Aus älteren Diskussionen meine ich aber mitgenommen zu haben, dass es dafür keinen wirklichen Ersatz gibt?).

Einen wirklich passenden Ersatz gibt es nicht da sämtliche Berechnungen auf Basis der Yahoo Weathercodes geschehen. Der Aufwand ist es einfach nicht wert denke ich. Persönlich ist meine Meinung eh Twilight sterben zu lassen und auf Astro aus zu weichen.

Grüße

yersinia · 03 September 2020, 14:04:50

Zitat von: CoolTux am 03 September 2020, 13:38:28Persönlich ist meine Meinung eh Twilight sterben zu lassen und auf Astro aus zu weichen.

Was _mir_ dann fehlen würde wären sx_indoor sowie sx_weather - dies kann mir weder Astro noch SUNRISE_EL zur Verfügung stellen.
MMn kann ich die Bewölkung im Twilight-Device mit dem Attribut useExtWeather von einem anderen Wetter Device Reading übernehmen - dann verschwindet auch die nervige Yahoo Warnung fast vollständig.

Zumal ich die Readings des Twilight-Devices insb. für Einsteiger sehr einfach finde - da ist dann alles dabei an Zeiten bezgl. sunrise und sunset. Ja, Astro bzw SUNRISE_EL können das auch, aber nicht derart komfortabel, imho.
Auch fand ich das Reading light praktisch, als es noch funktionierte.

Beta-User · 03 September 2020, 14:13:56

Hmm, ich kann das letztlich nicht beurteilen, da ich Astro einsetze (und gar keine Erfahrung mit Twilight habe), mich aber letztlich noch nie intensiver mit beiden auseinandergesetzt habe (nur mit bestimmten Code-Teilen, die sich Weekday- und RandomTimer daraus geborgt hatten).

Unschön ist zweierlei:
- Zum einen produziert es Logeinträge, von denen wir wissen, wo sie herkommen. Könnte man abstellen, die sind in der jetzigen Form für die User kein Mehrwert; ob man die zugehörigen Berechnungen/Herleitungen der Reading-Inhalte dann auch vereinfachen kann (wenn nicht erreichbar, wird mit defaults gerechnet, oder?), wäre eine weitere Frage.
- Der Status von Twilight ist nicht wirklich kommuniziert.

Das erste sollte man immer fixen, und WENN es so ist, dass man es ausphasen sollte, wäre das mindeste, in der cref (für neue User) und ggf. beim Start von FHEM (im Log) darauf hinzuweisen, dass man irgendwann auf einen passenden Ersatz (Astro? Sonst noch was, auch für Teile?) wechseln sollte, optimalerweise gleich mit einer Umstellungsroutine, falls sowas möglich ist (vorab schon mal sorry, wenn entsprechende Hinweise bereits erfolgen, ich habe nur in der d-commandref nachgesehen).

Dass man die Info finden kann, wenn man sucht und dort in dem 2019-er Thread ggf. auch Teillösungen und manches mehr (Danke auch für die Bereitstellung des Darksky-Plugins für weather!) diskutiert wurde, stimmt zwar, es ist aber mMn. kein guter Weg, irgendwie über einen "funktioniert nicht mehr"-Thread Module abzukündigen...

(Wenn es so ist, dass es "nur" darum geht, Twilight als Maintainer in Ehren zu beerdigen, kann ich das übernehmen, Umstellungsroutinen wird es dann aber vermutlich nicht geben...)

@yersinia:
Das sind m.E. gute Gründe. Falls du Interesse am Weiterleben von diesem Modul hast, können wir das auch zusammen in Co-Maintainerschaft übernehmen (und ich mich dann nach Ende der Überarbeitung dann wieder verabschiede). Es sollte halt so sein, dass wir wieder einen akzeptablen Stand bekommen und die Doku paßt...

CoolTux · 03 September 2020, 14:37:16

Wenn man sich die weitere Entwicklung auf Github an schaut so ist Twilight lediglich als Wrapper für Astro umgebaut worden und auf packages umgestellt.

https://github.com/fhem/Twilight/tree/development

Grüße

Christoph Morrison · 03 September 2020, 14:54:51

Zitat von: CoolTux am 03 September 2020, 13:18:42
Hat das Twilight Modul nicht der Christoph unter seinen Händen?

Ich habe alle meine Module kürzlich zurückgegeben (und auch meine neue Version von Twilight nie veröffentlicht, weil ich immer unzufrieden mit ihr war). Insofern, nein, weder igami (der sie vorher hatte) noch ich haben daran etwas gefixt.

Christoph Morrison · 03 September 2020, 15:00:18

Zitat von: CoolTux am 03 September 2020, 13:38:28
Einen wirklich passenden Ersatz gibt es nicht da sämtliche Berechnungen auf Basis der Yahoo Weathercodes geschehen. Der Aufwand ist es einfach nicht wert denke ich. Persönlich ist meine Meinung eh Twilight sterben zu lassen und auf Astro aus zu weichen.

Die Diskussion haben wir ja schon mal geführt (mit pah) und sie endete IIRC damit, dass Astro was anderes macht als Twilight und eine Mischung nicht gewünscht ist. Meine Idee war, Astro zur Berechnung der Daten zu nehmen, die man nicht für eine wetterabhängigen Daten braucht.

Beta-User · 03 September 2020, 15:01:57

Dann würde ich darum bitten, die angehängte Version mal zu testen.

Ist nicht viel anders, außer dass die yahoo betreffenden Log-outputs weg sind, beim Starten was geschrieben wird und die cref in Teilen angepaßt ist. Wir können dann später immer noch entscheiden, was denn jetzt letztendlich geschehen soll...

Beta-User · 03 September 2020, 16:46:13

...weil's grade so nett war, noch eine etwas erweiterte Version. Neben kleineren Änderungen in der cref ist v.a. der define-Teil geändert, Twilight ohne nähere Infos in der DEF greift damit direkt die Angaben ab, die man in global gemacht hatte (d.h., longitude und latitude wären damit auch optional, nicht mehr verpflichtend).

amenomade · 03 September 2020, 19:47:07

Astro funktioniert nicht auf allen Architekturen wegen POSIX::tzset

Zitat
Code Auswählen Erweitern
POSIX::tzset not implemented on this architecture at ./FHEM/95_Astro.pm line 2083.

Das heisst Twilight (als Wrapper für Astro) wird auch nicht mehr funktionieren?

Christoph Morrison · 03 September 2020, 20:09:02

Zitat von: amenomade am 03 September 2020, 19:47:07
Astro funktioniert nicht auf allen Architekturen wegen POSIX::tzsetDas heisst Twilight (als Wrapper für Astro) wird auch nicht mehr funktionieren?

Naja, noch hat niemand Astro in Twilight reinimplementiert. Was für eine Plattform hast du denn?

amenomade · 03 September 2020, 20:11:04

Meine Testinstanz :

ZitatThis is perl 5, version 26, subversion 3 (v5.26.3) built for MSWin32-x64-multi-thread
(with 2 registered patches, see perl -V for more detail)

Copyright 1987-2018, Larry Wall

Binary build 2603 [a95bce075] provided by ActiveState http://www.activestate.com
Built Dec 17 2018 09:46:45

Ich glaube, es ist das gleiche bei StrawberryPerl: https://forum.fhem.de/index.php?topic=102107.0

Bei der produktiven (Debian) geht es.

Beta-User · 04 September 2020, 09:37:41

Hmm, wenn es also (noch) keinen Sinn macht, Twilight zu beerdingen, würde ich Rudi anbieten, das als "orphan" interimsweise zu betreuen und wenigstens die Log-Einträge zu beseitigen etc., sofern positive Rückmeldung zu der letzten Testversion kommt? Ziel wäre dann aber eher eine Erhaltung des status quo, ggf. iVm. der Auslagerung einiger zentraler Mini-Funktionen.

Falls sich jemand anderes (mit) findet: feel free!

(@Christoph:
Ansonsten würde mich noch interessieren,
- ob es bei deinen Patchen irgendeinen (noch von Astro unabhängigen) Zwischenstatus gab, auf dem man aufsetzen kann und
- wie du den Hinweis auf InternalTimer gemeint hattest in irgendeinem früheren Thread (vermutlich zum RandomTimer-PBP-Review). Da ging es darum, ob diese "zentralen Mini-Funktionen" eigentlich überhaupt noch erforderlich sind. Ich hatte mir das im Nachgang angesehen, aber dann nicht verstanden, wie man diese "get-hash-indirect"-Geschichte direkt mit (Remove-)InternalTimer lösen könnte.)

Christoph Morrison · 04 September 2020, 10:57:57

Zitat von: Beta-User am 04 September 2020, 09:37:41
Ansonsten würde mich noch interessieren,
- ob es bei deinen Patchen irgendeinen (noch von Astro unabhängigen) Zwischenstatus gab, auf dem man aufsetzen kann und

Nur das, was es im development-Branch gibt, mehr habe ich daran nicht geändert.

Der Grund warum ich nicht an Twilight weiter entwickelt habe ist, dass mir die Situation mit der Abhängigkeit zu Astro nicht gefällt (kein Mensch, nicht mal pah, weiß ob Astro morgen noch das tut, was es gestern getan hat und es funktioniert auch nicht auf jeder Plattform, wie ich gestern gelernt habe), außerdem gibt es keinen Dependency-Mechanismus in update, also hätte Twilight eigentlich zwingend im SVN-Repository bleiben müssen, was mir aber auch nicht gefällt. Die Implementierung, die Dietmar seinerzeit gemacht hat, war wohl fehlerhaft, denn die Astro-Daten stimmen nicht mit den berechneten Daten anderer Portale überein, also hätte ich das alles nachimplementieren müssen, wozu mir die Zeit und Lust gefehlt hat.

Zitat von: Beta-User am 04 September 2020, 09:37:41
- wie du den Hinweis auf InternalTimer gemeint hattest in irgendeinem früheren Thread (vermutlich zum RandomTimer-PBP-Review). Da ging es darum, ob diese "zentralen Mini-Funktionen" eigentlich überhaupt noch erforderlich sind. Ich hatte mir das im Nachgang angesehen, aber dann nicht verstanden, wie man diese "get-hash-indirect"-Geschichte direkt mit (Remove-)InternalTimer lösen könnte.)

Ich hab leider keine Ahnung was du meinst. Soweit ich mich erinnere, wurde Twilight noch vor InternalTimer() geschrieben und bringt seinen eigene InternalTimer-Implementierung mit, die inzwischen ja überflüssig ist.