Neuauflage des Moduls 98_Siro.pm

[CoolTux]

Hallo Thomas,

Die oben aufgeführten Warnungen kommen daher, das Du das Value eines Hash's abfragst welchen Du aber immer wieder löschst.

Code Auswählen Erweitern


$hash->{helper}{progmode}


Code Auswählen Erweitern

$hash->{helper}{exexcmd}

Also ich würde empfehlen entweder nicht löschen oder vor dem abfragen des Values prüfen ob der Hash existent ist.

[Byte09]

Ich habe da noch was

Code Auswählen Erweitern


2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/98_Siro.pm line 623.
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/98_Siro.pm line 653.
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/98_Siro.pm line 679.
2019.05.31 08:50:13 3: Siro-Set (RolloWohnzimmer_F4) : set Up
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/98_Siro.pm line 291.
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value $args[1] in concatenation (.) or string at ./FHEM/98_Siro.pm line 298.
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value $args[1] in string eq at ./FHEM/98_Siro.pm line 300.
2019.05.31 08:50:13 1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/98_Siro.pm line 300.


hmm, ich habe jetzt eine andere version als du ( zeilennummern) . Denke aber das wird nur von einem LOG-befehl und nicht vorhhandenen hash verursacht bei nicht definierter variable - also nur unschön, nicht wild.

Code Auswählen Erweitern

Log3( $name, 5,"Siro_sendCommand: args1 - $args[1]");

Code Auswählen Erweitern

( $hash->{helper}{exexcmd} eq "off")


schaue ich mir aber im laufe des tages an .

gruss thomas

[Byte09]

Hallo Thomas,

Die oben aufgeführten Warnungen kommen daher, das Du das Value eines Hash's abfragst welchen Du aber immer wieder löschst.

Code Auswählen Erweitern


$hash->{helper}{progmode}


Code Auswählen Erweitern

$hash->{helper}{exexcmd}

Code Auswählen Erweitern

$hash->{helper}{exexcmd}


Also ich würde empfehlen entweder nicht löschen oder vor dem abfragen des Values prüfen ob der Hash existent ist.

dieses ist in der aktuellsten version bereits behoben , der andere noch nicht

gruss thomas

[CoolTux]

Ich habe gerade die aktuelle besorgt und gehe die Fehlermeldungen durch.
Ich denke es liegt unter anderem daran das mein Rolllo oben ist und ich ihm sage er soll hoch fahren.

Ich mache mal die Fehler weg und schicke dir dann ein patch. Ausgehend von der aktuellen Version im Github

[CoolTux]

Hallo Thomas,

Anbei ein getesteter Patch. Die Warnings sind nun nicht mehr vorhanden.


Grüße


WARNUNG AN DIE USER: Bitte diesen Patch nicht eigenmächtig anwenden. Wartet bis der Modulauthor ihn eingebaut hat und die neue Modulversion veröffentlicht.

[Byte09]

Hallo Thomas,

Anbei ein getesteter Patch. Die Warnings sind nun nicht mehr vorhanden.


Grüße


WARNUNG AN DIE USER: Bitte diesen Patch nicht eigenmächtig anwenden. Wartet bis der Modulauthor ihn eingebaut hat und die neue Modulversion veröffentlicht.

super, dank dir.

habe das bei mir eben eingebaut und werde es heute nachmittag nochmal in das svn schieben.

gruss thomas

[volschin]

Was ist denn aus last_reset_os und operating_seconds geworden?
Ich hatte mir daraus so eine schöne Akkustandswarnung gebaut, die jetzt nicht mehr funktioniert. [emoji853]

[Byte09]

Was ist denn aus last_reset_os und operating_seconds geworden?
Ich hatte mir daraus so eine schöne Akkustandswarnung gebaut, die jetzt nicht mehr funktioniert. [emoji853]

aus 'operating_seconds' ist 'motor-term' geworden,'last_reset_os' habe ich nicht mehr drinnen , kann ich aber einbauen , wenn du es benötigst.

das attribut 'SIRO_Battery_low' ( angabe sekunden ) bewirkt , dass nach der motorlaufzeit x dea reading 'batteryState' auf low gesetzt wird.

wenn du 'last_reset_os' trotzdem benötigst gib mir kurz bescheid, kann ich dannsehr kurzfristig ändern.

gruss Thomas

[volschin]

Ich benötige es definitiv, da ich festgestellt habe, dass ein nicht unwesentlicher Teil der Leistung auch im Standby abfließt. Damit ist eine Berechnung rein nach der Motorfahrzeit nicht aussagefähig.

[Byte09]

Ich benötige es definitiv, da ich festgestellt habe, dass ein nicht unwesentlicher Teil der Leistung auch im Standby abfließt. Damit ist eine Berechnung rein nach der Motorfahrzeit nicht aussagefähig.

stelle es in 10 minuten in das svn , das reading wird aber 'motor-term-reset' heissen - inhalt unixtime .

gruss thomas

[Byte09]

anbei die version incl 'motor-term-reset'

ab morgen dann auch im fhemupdate

gruss thomas

[Byte09]

das reading beinhaltet nur die unixtime des letzten reset. wenn du es im anderen format , anzahl der tage , etc. benötigst sollte daraus alles per userreading machbar sein.

kommst du damit so klar ?

gruss thomas

[volschin]

Super, danke. Die Umrechnung von unixtime auf Dauer bekomme ich noch hin. [emoji6]

[Byte09]

Super, danke. Die Umrechnung von unixtime auf Dauer bekomme ich noch hin. [emoji6]

denke ich mir  , habe es eben bei mir auch mal als userreading eingebaut:

Code Auswählen Erweitern

days-since-reset {my $lastreset = ReadingsVal( $name, 'motor-term-reset', time );my $secs = (time - $lastreset)/86400;my @out = split (/\./,$secs); return $out[0];}

gruss thomas

[volschin]

Bei meinen ERB16LE funktioniert das fav nicht mehr.
Außerdem lassen sich die Rollos aus HomeKit nicht mehr fahren und zeigen auch nicht den korrekten Status. Auch das DevStateicon ändert sich nicht mehr selbst von RunningDown auf Geschlossen. Da scheint kein Event getriggert zu werden.

Code Auswählen Erweitern

2019.06.01 16:33:32 1: Siro-Set: Favoritenanfahrt nicht m?glich , Reading nicht gesetzt