Wunschliste - Patch für 91_sequence.pm

Begonnen von RomanticBoy83, 07 Juli 2021, 18:25:49

Vorheriges Thema - Nächstes Thema

RomanticBoy83

Hallo in die Runde,
... ich möchte hier einen Patch als Vorschlag zur Übernahme posten.

Zum Problem:
Ich möchte gerne eine "Sequence" nutzen um eine Tastatureingabe als Codeschloss zu verwenden. Leider sind Fehleingaben/Versuche so nicht zu realisieren.
Als Beispiel möchte ich nur auf die Sequenz 1;2;3;4 reagieren. Sollte jedoch jemand sehr schnell alle Tasten Drücken, so würde die "Sequence" nach viermaligen "alle-Tasten-gedrückt" natürlich auch triggern.

Zur Lösung:
Ich habe die "Sequence" um das optionale Attribut strictSequence erweitert. Sollte dieses auf 1 gesetzt sein, so wird die "Sequence" stringent auf die richtige Reihenfolge der Events achten. Sollte ein unerwartetes Event kommen, so wird die "Sequence" sofort abgebrochen und die Eingabe muss erneut von Begin an erfolgen.

Meine Intension ist es, dass der Patch eventuell vom rudolfkoenig als Author übernommen wird. Sollte es Gründe geben für ein nicht Übernehmen, welche durch fehlende Arbeiten begründet sind, dann werde ich diese natürlich gerne nachreichen.

betateilchen

Tipp: Verschiebe den Thread in das für das Modul richtige Unterforum ("Automatisierung"), dann hast Du größere Chancen, dass Rudi Deinen Vorschlag entdeckt. Wunschlisten werden gerne mal übersehen.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

Beta-User

Über verschobene Threads wird Rudi nicht automatisch benachrichtigt.
Besser nochmal einen neuen Hinweis-Thread aufmachen - zumindest, wenn in den nächsten paar Tagen keine Reaktion kam.
Server: HP-elitedesk@Debian 12, aktuelles FHEM@ConfigDB | CUL_HM (VCCU) | MQTT2: MiLight@ESP-GW, BT@OpenMQTTGw | MySensors: seriell, v.a. 2.3.1@RS485 | ZWave | ZigBee@deCONZ | SIGNALduino | MapleCUN | RHASSPY
svn: u.a MySensors, Weekday-&RandomTimer, Twilight,  div. attrTemplate-files

rudolfkoenig

Habe den Vorschlag (angepasst) uebernommen, und in der Doku auf die Nebeneffekte hingewiesen:
ZitatNote: it will only work as correctly if the sequence is triggering only to the configured devices (check the NOTIFYDEV internal), and the source is not generating "undesired" events together with the desired ones.

mebwaster

Habe heute aktualisiert, danach fhem neugestartet. Das blieb dann bei 100% Auslastung beim Starten stehen.
In der debug-Ausgabe lief das in Endlosschleife:

2023.07.14 16:12:24 5: sequence wz_Rollo1_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo1_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo234_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: Starting notify loop for wz_Rollo234_Ab_KurzLang, 1 event(s), first is partial_0
2023.07.14 16:12:24 5: sequence df_Bernhard_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence df_Bernhard_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauHauptlichtVerstaerker abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauKanalAbLeiser abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauKanalAufLauter abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: Starting notify loop for remoteBlauKanalAufLauter, 1 event(s), first is partial_0
2023.07.14 16:12:24 5: sequence df_Bernhard_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence df_Bernhard_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauHauptlichtVerstaerker abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauKanalAbLeiser abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence remoteBlauKanalAufLauter abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo1_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo1_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo234_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo234_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: End notify loop for remoteBlauKanalAufLauter
2023.07.14 16:12:24 5: sequence wz_Rollo1_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo1_Auf_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo234_Ab_KurzLang abort-strict on 0 (partial_0)
2023.07.14 16:12:24 5: sequence wz_Rollo234_Auf_KurzLang abort-strict on 0 (partial_0)


Habe dann komplett auf die vorige Sicherung zurückgestellt, da funktionierts dann wieder. Da es sich bei den obigen Objecten um sequence's handelt, vermute ich daß das mit dieser Änderung zusammenhängt?

rudolfkoenig

ZitatHabe heute aktualisiert, danach fhem neugestartet. Das blieb dann bei 100% Auslastung beim Starten stehen.
Da ist mir offensichtlich beim Anpassung des Patches ein Fehler unterlaufen.
Habs gefixt und eingecheckt.