Neuauflage des Moduls 98_Siro.pm

Begonnen von Byte09, 17 Mai 2019, 06:06:34

Vorheriges Thema - Nächstes Thema

Byte09

Zitat von: Steeve23 am 30 Juni 2019, 12:32:37
Habe ich gerade getestet, der Motor quittiert den Einstellmodus, er stopt auch wenn ich "stop" drücke, allerdings auch nach mehrmaligen "stop" drücken kommt keine quittierung vom Antrieb.

zeig mir bitte mal ein list des devices

gruss thomas

Steeve23

Zitat von: Byte09 am 30 Juni 2019, 12:44:39
zeig mir bitte mal ein list des devices

gruss thomas


Zitat
So sieht das bei mir aus

Internals:
   CHANNEL_RECEIVE 1
   DEF        A23B7C41
   FUUID      5cf23faf-f33f-ade1-98d3-812ef93bcd81eb54
   ID         A23B7C4
   IODev      SIGNALDuino
   MODEL      LE-Device
   NAME       Rollo
   NR         225
   NTFY_ORDER 50-Rollo
   STATE      0
   TYPE       Siro
   Version    1.3
   .attraggr:
   .attrminint:
   CODE:
     1          A23B7C41
   READINGS:
     2019-06-30 12:08:32   ActionTime      1561889312
     2019-06-30 12:08:32   ActionTrigger   fhem
     2019-06-30 12:08:32   BetweentActionTime 19
     2019-06-30 12:05:02   Favorite-Position 94
     2019-06-30 12:08:13   LastAction      off
     2019-06-30 12:08:32   LastActionTime  1561889293
     2019-06-30 12:08:17   aktActionFinish 0
     2019-06-30 12:08:17   aktEndAction    0
     2019-06-30 12:08:17   aktRunningAction noAction
     2019-06-30 12:08:17   aktTimeAction   0
     2019-06-30 12:08:32   batteryState    unknown
     2019-06-30 12:08:17   motor-term      2036.47000279427
     2019-06-07 13:21:17   motor-term-reset 1559906477
     2019-06-30 12:08:17   pct             0
     2019-06-30 12:08:17   position        0
     2019-06-30 12:08:17   state           0
   helper:
     exexcmd    on
     ignorecmd  off
     progmode   off
Attributes:
   DbLogExclude .*
   IODev      SIGNALDuino
   SIRO_signalLongStopRepeats 45
   SIRO_signalRepeats 1
   SIRO_time_to_close 23
   SIRO_time_to_open 23
   alexaName  Rollo
   alexaRoom  Schlafzimmer
   devStateIcon {if (ReadingsVal( $name, 'state', 'undef' ) =~ m/[a-z]/ ) { return 'programming:edit_settings notAvaible:hue_room_garage runningUp.*:fts_shutter_up runningDown.*:fts_shutter_down'}else{return '[0-9]{1,3}:fts_shutter_1w_'.(int($state/10)*10)}}
   genericDeviceType blind
   room       4-Schlafzimmer,alexa
   verbose    0
   webCmd     prog:stop:open:close:fav:pct

Byte09

ich bin mir noch nicht ganz sicher , ggf. ein fehler im modul.

kannst du bitte mal probieren ob der motoer ( modul im programmiermodus ) einmal oder zeimal quittiert, wenn du nur den butto P2 drückst ?

gruss thomas

Steeve23

Also ich habs getestet , bei drücken auf P2 reagiert der Antrieb mit 2x quittieren

Byte09

Zitat von: Steeve23 am 30 Juni 2019, 13:21:18
Also ich habs getestet , bei drücken auf P2 reagiert der Antrieb mit 2x quittieren

darf nicht sein , muss nur einmal quittieren.

habe eben eine korrektur im modul gemacht und dieses in das GIT gestellt.

gib bitte mal in der befehlszeile folgendes ein:
update all https://raw.githubusercontent.com/Byte009/FHEM-Siro/master/controls_siro.txt
danach
reload 98_Siro.pm

... und versuch das ganze dann bitte nochmal.

gruss thomas

Steeve23

Hallo Thomas,
also der Antieb hat die Enlagen jetzt gespeichert, und läuft Perfekt,
vielen Dank für den tollen Support.

Das einzige was jetzt nach dem Update auftritt sind jede menge Perl Wsrnungen im Log.
Zitat2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 1413.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 1351.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 1296.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 1247.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 1011.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 984.
2019.06.30 13:49:20 1: PERL WARNING: Use of uninitialized value $comand in string eq at ./FHEM/98_Siro.pm line 971.
2019.06.30 13:46:42 1: PERL WARNING: Use of uninitialized value $zielposition in concatenation (.) or string at ./FHEM/98_Siro.pm line 1104.
2019.06.30 13:44:51 1: PERL WARNING: Use of uninitialized value $cmd in concatenation (.) or string at ./FHEM/98_Siro.pm line 443.
2019.06.30 13:44:51 1: PERL WARNING: Use of uninitialized value $command in hex at ./FHEM/98_Siro.pm line 433.
2019.06.30 13:44:51 1: PERL WARNING: Use of uninitialized value $args[0] in join or string at ./FHEM/98_Siro.pm line 423.
2019.06.30 13:44:51 1: PERL WARNING: Use of uninitialized value $cmd in hash element at ./FHEM/98_Siro.pm line 383.

Byte09

Zitat von: Steeve23 am 30 Juni 2019, 13:57:53
Hallo Thomas,
also der Antieb hat die Enlagen jetzt gespeichert, und läuft Perfekt,
vielen Dank für den tollen Support.

Das einzige was jetzt nach dem Update auftritt sind jede menge Perl Wsrnungen im Log.

ja, muss zugeben dass ich nur dieses fehlverhalten schnell gefixt habe und selber nicht einen blick in das log geworfen habe. ich schaue mir das im laufe des tages noch an , und behebe das. sollte dann mit dem morgigen normalem update weg sein.

gruss thomas

Hardlife

Hallo,

erstmal danke für all die Arbeit, die in dem neuen Modul steckt.
Bringt wirklich erhebliche Verbesserungen mit sich.

Eine Sache ist mir beim "Kanal 0" (alle Rollos gleichzeitig ansprechen) aufgefallen:
-> Leider funktioniert das senden der "Favoriten-Position" wohl nicht ganz korrekt?

Wenn ich "set Siro_XXX_Kanal0 fav" auslöse, werden zwar in den FHEM-Devices die Aktionen gesetzt und in der FHEM-Oberfläche stellen sich all meine Rollos auf die programmierten Prozentwerte....
Aber die Motoren reagieren nicht...
Wird der Befehl nicht gesendet?

Wenn ich den Befehl für "open" oder "close" sende, funktioniert alles wunderbar.
In den FHEM-Devices wird der Status gesetzt und auch die Motoren reagieren.

Favoriten sind natürlich neu gesetzt worden und die Werte sind in den Internals der einzelnen Rollos vorhanden.
Wenn ich im Einzel-Device (z.B.: Siro_Rollo_Wohnzimmerfenster) den "fav" Befehl auslöse, funktioniert das auch...
Nur eben nicht über das "Kanal 0"-Device

Hat das Problem vielleicht sonst noch wer?


Danke vorab für die Mühe,
Hardlife
Raspi 4B
nanoCUL-868 & 433,JeeLink,milight,Signalduino,GPIO-433er-Sender/Empfänger, GPIO-Infrarot,GSM-Stick für SMS
MAX!-Heizungssteuerung,Intertechno-V1-Steckdosen + V3-Dimmer,"Flamingo FA21RF"-Funk-Rauchmelder
433er-China-Bewegungsmelder,"Voltcraft CO20"-Stick,LaCrosse-Temperatur,Revolt-NC5462

Byte09

#233
hi hardlife,

klingt mir doch sehr nach einem Bug im Modul. Ich schaue mir das an, komme aber leider erst morgen abend oder Montag dazu.

gruss Byte09


Gesendet von meinem ELE-L29 mit Tapatalk


Hardlife

Super, danke für die rasche Reaktion.
Raspi 4B
nanoCUL-868 & 433,JeeLink,milight,Signalduino,GPIO-433er-Sender/Empfänger, GPIO-Infrarot,GSM-Stick für SMS
MAX!-Heizungssteuerung,Intertechno-V1-Steckdosen + V3-Dimmer,"Flamingo FA21RF"-Funk-Rauchmelder
433er-China-Bewegungsmelder,"Voltcraft CO20"-Stick,LaCrosse-Temperatur,Revolt-NC5462

Byte09

#235
Zitat von: Hardlife am 07 Juli 2019, 02:07:39
Super, danke für die rasche Reaktion.

ich habe eben ein Update in das SVN gespielt, sollte noch für den heutigen durchgang reichen. Wenn du also nach 08.00 uhr ( wenn die heutige aktualisierung durch ist ) ein update machst sollte das Problem behoben sein.

Achte bitte darauf, das im device ( kanal0 ) das attribut 'SIRO_signalLongStopRepeats' ausreichend hoch gesetzt ist , stell es ruhig auf '45'.

Gib mir doch bitte kurz bescheid, ob es nun geht - ich habe das heute morgen recht schnell gefixt um es noch in das heutige update zu bekommen und konnte nicht gross testen.

gruss Byte09

Hardlife

Hi Byte09,

vielen Dank für Dein Engagement und den schnellen Fix.
Hab´s eben durchgetestet, funktioniert nun alles.  :)

Nun werkeln meine Rollos wieder vollautomatisch dahin....

LG,
Hardlife
Raspi 4B
nanoCUL-868 & 433,JeeLink,milight,Signalduino,GPIO-433er-Sender/Empfänger, GPIO-Infrarot,GSM-Stick für SMS
MAX!-Heizungssteuerung,Intertechno-V1-Steckdosen + V3-Dimmer,"Flamingo FA21RF"-Funk-Rauchmelder
433er-China-Bewegungsmelder,"Voltcraft CO20"-Stick,LaCrosse-Temperatur,Revolt-NC5462

Kuehnhackel

Hallo,

ich wollte mich auch mal für das tolle Modul und die Arbeit bedanken.

devStateIcon = {return FHEM::Siro::Siro_icon($name)}

Ich bekomme im offenen Zustand kein Icon angezeigt. Woran kann das liegen?

Gruß Ralf

det.

Zitat von: Kuehnhackel am 07 Juli 2019, 20:41:11

Ich bekomme im offenen Zustand kein Icon angezeigt. Woran kann das liegen?

Gruß Ralf
Das liegt am Icon, offen ist nichts vom Rollo zu sehen, da nur der Behang ohne Rand dargestellt ist
LG
det.

Byte09

#239
Zitat von: Kuehnhackel am 07 Juli 2019, 20:41:11
Hallo,

ich wollte mich auch mal für das tolle Modul und die Arbeit bedanken.

devStateIcon = {return FHEM::Siro::Siro_icon($name)}

Ich bekomme im offenen Zustand kein Icon angezeigt. Woran kann das liegen?

Gruß Ralf

wie det schon sagte ist das icon für den geöffneten rollo zwar vorhanden , aber im grunde ist es ein leeres feld.

wenn du einen entsprechenden iconsatz hast ( in entsprechendem ordner ) kannst du diese Information im devstateicon mitgeben , und dieser iconsatz wird genutzt .
devStateIcon = {return FHEM::Siro::Siro_icon($name,'iconname')}

das vorhandensein folgender icons wird dann erwartet:
iconname0 - geöffnet
iconname10 - 10 prozent geschlossen
iconname20 - 20 prozent geschlossen
.
.
.
iconname100 - 100 prozent geschlossen

andenfalls müsstest du dir das devstateicon komplett selber bauen im attribut . Die folgenden Zeilen sind aus der routine, die im devstateicon aufgerufen wird, da kannst du alle benötigten infos rausziehen.

my $sticon = "fts_shutter_1w_";
$sticon = $icon if defined $icon;


my $invers = AttrVal( $name, 'SIRO_inversPosition',0 );
my $ret ="programming:edit_settings notAvaible:hue_room_garage runningUp.*:fts_shutter_up:stop runningDown.*:fts_shutter_down:stop ".$state.":".$sticon.(int($state/10)*10).":".$move;
$ret ="programming:edit_settings notAvaible:hue_room_garage runningUp.*:fts_shutter_up:stop runningDown.*:fts_shutter_down:stop ".$state.":".$sticon.(100 - (int($state/10)*10)).":".$move if $invers eq "1";
$ret =".*:fts_shutter_all" if ($hash->{CHANNEL_RECEIVE} eq '0');
$ret =".*:secur_locked\@red" if ReadingsVal( $name, 'lock_cmd', 'off' ) eq 'on';


gruss Byte09