Gelöst: Fehler in 59_Weather.pm

[Pjg]

Ich erwarte hier echt keinen professionellen Support. Obwohl es hier schon oft eher besser ist als bei so manchem was man bei Profis bekommt.

Ich bin hier mal irgendwo über eine Liste der Modulowner gestoßen, mit den Unterforen wo sie aktiv sind.
Dann könnte ich das Thema da platzieren.

Beim define kann man ja nicht allzu viel falsch machen, oder?

define MyWeather Weather 999999 3600 de
attr MyWeather alias Yahoo Weather Musterstadt
attr MyWeather verbose 0

Da fällt mir ein. Hast Du die aktuelle Version?
Da wurde zuletzt noch was geändert wegen Wettericons.
Evtl. ist da ja etwas schief gelaufen.

[betateilchen]

Ich bin hier mal irgendwo über eine Liste der Modulowner gestoßen, mit den Unterforen wo sie aktiv sind.
Dann könnte ich das Thema da platzieren.

Das Thema steht hier genau im richtigen Unterforum.

Code Auswählen Erweitern

FHEM/59_Weather.pm           borisneubert         http://forum.fhem.de Unterstützende Dienste

Beim define kann man ja nicht allzu viel falsch machen, oder?

Nein, beim Define kann man nicht viel falschmachen.

Aber mit Deinem "Muster-Define" kann ich nix anfangen, denn ich wollte tatäschlich Dein ECHTES Define haben, damit mit DEINER genutzten Wetterstation testen kann, denn mit meiner genutzten Yahoo-Station kommen die Meldungen nicht.

Ich würde Dir ja gerne helfen - aber solange ich die Yahoo-Id nicht kenne, die Du verwendest, ist das schwer möglich.

Da fällt mir ein. Hast Du die aktuelle Version?

Um den Update-Status meiner fhem-Installation musst Du Dir keine Gedanken machen. Und mit Wetter-Icons haben Deine Meldungen nix zu tun, sondern mit der Verarbeitung der Wind-Daten.

[Pjg]

Dachte nur das dabei evtl. auch etwas anderes angepasst wurde was zu demFehler führt.
Die ID habe ich Dur geschickt.
Danke.

[betateilchen]

Die ID habe ich Dur geschickt.

Genau wie ich vermutet hatte: Deine Wetterstation liefert keine Winddaten:

(http://up.picr.de/19906793ge.png)

Alles was da unter "Wind" steht, ist deshalb nicht zu gebrauchen. Dieser Fall ist im Modul nicht komplett abgefangen.
Man sollte die Wind-Readings in diesem Fall gar nicht erst erzeugen.

[betateilchen]

@Boris:

getesteter Patchvorschlag, der die Warnungen vermeidet:

Code Auswählen Erweitern


Index: 59_Weather.pm
===================================================================
--- 59_Weather.pm (Revision 6804)
+++ 59_Weather.pm (Arbeitskopie)
@@ -361,11 +361,11 @@
       if ($tag eq "yweather:wind" ) {
         $value =~/chill="(-?[0-9.]*?)" .*direction="([0-9.]*?)" .*speed="([0-9.]*?)" .*/;
         $urlResult->{"readings"}->{"wind_chill"} = $1 if (defined($1));
-        $urlResult->{"readings"}->{"wind_direction"} = $2 if defined($2);
-        my $windspeed= defined($3) ? int($3+0.5) : "";
+        $urlResult->{"readings"}->{"wind_direction"} = $2 if ($2 ne "");
+        my $windspeed= ($3 ne "") ? int($3+0.5) : "n/a";
         $urlResult->{"readings"}->{"wind_speed"} = $windspeed;
         $urlResult->{"readings"}->{"wind"} = $windspeed;# duplicate for compatibility
-        if (defined($2) & defined($3)) {
+        if ($2 ne "" && $3 ne "") {
           my $wdir = degrees_to_direction($2,@directions_txt_i18n);
           $urlResult->{"readings"}->{"wind_condition"} = "Wind: $wdir $windspeed km/h"; # compatibility
         }


[Dr. Boris Neubert]

Die Meldung kannst Du nicht abschalten.

Keine Sorge, die Modulmaintainer lesen hier im Forum regelmäßig mit und kümmern sich um die Meldungen. Aber wir betreiben das alles nur als Hobby und haben nebenbei vielleicht auch noch eine Familie zu bespaßen und einen Job, mit dem wir unser Geld verdienen. Also erwarte bitte keinen 24/7 Support.

Ignoriere die Meldungen einfach bis auf weiteres, irgendwann gibts die nicht mehr, weil die Problemstellen im Coding behoben sind.

FULL ACK.

Ist aber bald Weihnachten.

Ups, habe ich viele Module...

Aber ich habe ja nicht gesagt, welches Jahr 

Grüße
Boris

[betateilchen]

Ich bin ja schon dabei, Dir Vorarbeit zu leisten, weil ich im gds und im configdb bisher keine solche Warnungen produziere und somit "nix zu tun" habe 

In den Modulen von Dir, die ich auch selbst benutze, kenne ich mich ja inzwischen recht gut aus

[Dr. Boris Neubert]

Ich bin ja schon dabei, Dir Vorarbeit zu leisten

Danke. Ich bin allerdings derzeit ein paar tausend Kilometer von meinem Entwicklungsrechner entfernt.

Viele Grüße
Boris

[betateilchen]

Bei Bedarf kann ich die vorgeschlagene Änderung für das Wettermodul als Vorab-Fix einchecken.

[Pjg]

Danke für das Coding.
Ich ändere es bei mir ab.
24/7 habe ich nie erwartet, aber auf "Weihnachten-in-ferner-Zukunft-Sprüche"
habe ich auch keine Lust.

Danke betateilchen.

[Dr. Boris Neubert]

Bei Bedarf kann ich die vorgeschlagene Änderung für das Wettermodul als Vorab-Fix einchecken.

Ich denke, dass das nicht nötig ist. Der OP hat sich schon selbst geholfen.

Ich warte erst einmal ab, welche Warnungen noch hier auflaufen, und werde mich dann mal an einem Wochenende gebündelt damit befassen. Ich sehe bei mir sowohl bei meinen als auch bei fremden Modulen zahlreiche Warnungen im Log. Die Priorität ist für mich allerdings nicht hoch und der Termin Weihnachten ist durchaus realistisch.

Viele Grüße
Boris