FHEM Forum

Ich weiß nicht, ob das das richtige Unterforum ist.
Aber seit dem letzten update erhalte ich folgende 2 Fehlermeldungen stündlich ins log:

2014.10.23 09:53:01 1: PERL WARNING: Argument "" isn't numeric in addition (+) at ./FHEM/59_Weather.pm line 365.
2014.10.23 09:53:01 1: PERL WARNING: Argument "" isn't numeric in addition (+) at ./FHEM/59_Weather.pm line 173.
2014.10.23 10:53:01 1: PERL WARNING: Argument "" isn't numeric in addition (+) at ./FHEM/59_Weather.pm line 365.
2014.10.23 10:53:01 1: PERL WARNING: Argument "" isn't numeric in addition (+) at ./FHEM/59_Weather.pm line 173.

Das sind erstmal überhaupt keine Fehlermeldungen, sondern nur Warnung - steht auch im Text.

Ausserdem ist das "Problem" schon seit zwei oder drei Wochen bekannt und es geht da nicht nur um das Wetter-Modul. Fhem macht seit neuestem eine sehr viel striktere Fehlerprüfung in den Modulen und gibt seither solche "Schönheitsfehler" als Warnung aus, damit sich hoffentlich irgendwann jemand darum kümmert.

Funktionieren tut das Wettermodul aber ungeachtet dieser Meldung trotzdem.

verbose = 0

Hallo,

Nicht verbose sondern stacktrace abschalten.
Dann verschwinden auch die deutlichen Warnhinweise wieder.

Grüße

Nein, tun sie nicht, die Warnhinweise bleiben auch bei stacktrace=0.

Nur das stacktrace verschwindet - genau wie der Attributname vermuten läßt ;)

Hallo,

Zitatdie Warnhinweise bleiben auch bei stacktrace=0.

Ah, ok.
Da ich diese Warnhinweise dank meiner alten Installation nicht habe (oder nicht sehe  ;D ) bin ich wohl fälschlicherweise davon ausgegangen das bei stacktrace=0 auch die deutlichen Hinweise wieder verschwinden würden.

Aber ich lerne immer gerne noch dazu  8)

Grüße

Du hättest bei genauem Lesen des Eingangsbeitrages auch selbst draufkommen können, dass stacktrace gar nicht aktiviert ist ;)

Hallo,

mensch betateilchen - lass mich doch nicht dumm sterben.
Ich hab den Eingangsbeitrag jetzt 3-mal angeschaut und durchgelesen - nochmal und ich nehm ne Lupe  ???
Woran bitte sollte ich dort erkennen das stacktrace deaktiviert ist?

Grüße

Nichts einfacher als das. Guckst Du:

Zitat von: Pjg am 23 Oktober 2014, 12:46:02
Aber seit dem letzten update erhalte ich

also...

• weil stacktrace standardmäßig nach dem fhem.pl Update deaktiviert ist
• weil der Fragesteller nichts davon geschrieben hat, dass er stacktrace aktiviert hat
• weil im Logauszug überhaupt kein stacktrace zu sehen ist, sondern nur perl Warnungen

(ich hoffe, ich hab keinen Grund vergessen)

Hallo,

was aber nur Annahmen sind da
a) wir nicht wissen wann das letzte Update gemacht wurde
b) Fragesteller so Kleinigkeiten gerne mal vergessen und
c) gerne auch Logfileauszüge gefiltert werden

Du siehst - ich bin von ganz anderen Voraussetzungen aus gegangen.
Und ich dachte schon ich bin blind  ::)

Grüße

P.S.: Das stacktrace per default im letzten Update wieder deaktiviert ist weiß ich  8)

a) ist irrelevant, da es auf jeden Fall eine Version von fhem.pl war, in der die Meldungen überhaupt ausgegeben werden
b) bei einem Fragesteller mit einer solchen Frage gehe ich nicht davon aus, dass er uns was veschwiegen hat
c) Da ist für mich nichts gefiltert (hängt auch mit b) zusammen und resultiert aus langjähriger Supporterfahrung)


Aber langsam werden wir offtopic und Du wirst Dein Verhalten bezüglich vorschneller stereotyper Antworten vermutlich ohnehin nicht mehr ändern  8)

Zitatund Du wirst Dein Verhalten bezüglich vorschneller stereotyper Antworten vermutlich ohnehin nicht mehr ändern

Dazu bin ich vermutlich schon zu alt  :P
Und nun BTT

Ziemlich hartnäckig eure Meldung.
verbose=0 im weather device bringt es auch nicht.

Wäre es generell nicht irgendwie sinniger die modulowner auf die Schönheitsfehler hinzuweisen statt den Anwendern der Module das log voll zu schreiben?

Wie bekomme ich das denn wieder raus?

Jetzt hab ich schon den nächsten
59_Twilight

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.

Bei mir taucht die Meldung nicht in den Logs auf.

Kannst Du mal bitte das define Deines yahoo-Wetters in fhem posten?

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?

Zitatdefine 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.

Zitat von: Pjg am 24 Oktober 2014, 08:11:16
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.

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

Zitat von: Pjg am 24 Oktober 2014, 08:11:16
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.

Zitat von: Pjg am 24 Oktober 2014, 08:11:16
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.

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

Zitat von: Pjg am 24 Oktober 2014, 10:54:24
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.

@Boris:

getesteter Patchvorschlag, der die Warnungen vermeidet:


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
         }


Zitat von: betateilchen am 23 Oktober 2014, 20:56:55
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  8)

Grüße
Boris

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  :P

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

Zitat von: betateilchen am 24 Oktober 2014, 20:24:38
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

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

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.

Zitat von: betateilchen am 24 Oktober 2014, 21:07:51
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