[Vorlage] Variable Zeitsteuerung mit VIS Editor

sigi234

@GiuseppeS sagte in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

1x Editor-View

Habs jetzt begriffen.

gender

@GiuseppeS
Erstmal vielen Dank für die schnelle Aufnahme meines Feature-Request bezüglich der Random-Zeit, nachdem eine Bedingung nachträglich erfüllt ist.

Habe das natürlich gleich getestet und folgendes (falsches) Verhalten ist mir aufgefallen:
Grundsätzlich wird, nachdem eine Bedingung nachträglich erfüllt wurde, nach einer Random-Zeit die Aktion tatsächlich ausgeführt (sagen wir nach Random-10 Minuten soll Rollade A runterfahren). Sehr gut.
Das hat bei mir auch funktioniert. Allerdings wurde dann später, nach dem die Rollade bereits runtergefahren war, nochmal versucht, die Rollade runterzufahren (obwohl sie schon runtergefahren war).

Der Log sagt, dass in der Zeit, in der der (nachträgliche) Random bereits läuft, die Überprüfung, ob ein Random-Wert zum Runterfahren bestimmt werden soll, weiterhin bzw. nochmals passiert. Das hat zur Folge, dass weitere Random-Zeiten zum Runterfahren berechnet werden, solande die jeweilige Rollade nicht tatsächlich runtergefahren ist. Und das bedeutet dann natürlich, dass irgendwann die Rollade runtergefahren ist und dann dies aber nochmals passiert (da die weitere Random-Timer dann später durchgeführt werden).

Hier ein Auszug aus meinem Log für zwei Rolladen: Bei der Rollade "Arbeitszimmer" wird zweimal der Timer ausgeführt. Bei der Rollade "Wohnzimmer" wird kein weiterer Timer gesetzt, da hier die berechnete Random-Zeit nur 1 Minute betragen hat.
Wie wird denn die Berechnung getriggert? Ich nehme an, sobald sich der Wert der gesetzten Bedingung ändert. Das ist bei mir ja ein Helligkeitssensor, der natürlich dann öfters triggert.

* 2020-07-17 22:03:45.378  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Arbeitszimmer.Rollade:1.LEVEL (0) -> Timer wird in 4 Minuten ausgeführt. Bedingung(en) nachträglich erfüllt!
* 2020-07-17 22:03:45.383  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Wohnzimmer.RolladeRechts:1.LEVEL (0) -> Timer wird in 1 Minuten ausgeführt. Bedingung(en) nachträglich erfüllt!

* 2020-07-17 22:04:45.786  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Wohnzimmer.RolladeRechts:1.LEVEL (0) -> Timer ausgeführt. Bedingung(en) nachträglich erfüllt!

%(#ff0000)[* 2020-07-17 22:06:26.167  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Arbeitszimmer.Rollade:1.LEVEL (0) -> Timer wird in 14 Minuten ausgeführt. Bedingung(en) nachträglich erfüllt!]

* 2020-07-17 22:07:45.383  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Arbeitszimmer.Rollade:1.LEVEL (0) -> Timer ausgeführt. Bedingung(en) nachträglich erfüllt!
* 2020-07-17 22:20:26.168  - [32minfo[39m: javascript.0 (506) script.js.common.VIS.RolladenCtrl: Timer: Arbeitszimmer.Rollade:1.LEVEL (0) -> Timer ausgeführt. Bedingung(en) nachträglich erfüllt!

GiuseppeS

@gender
Diesen Fall habe ich tatsächlich nicht bedacht. Dass die Bedingung wahr wird, dann falsch und bevor der Random durch ist, wieder wahr.

Ein Grund für dieses Verhalten ist wie folgt:
Ich hatte bedacht, wenn die Bedingung true ist und bis zum Ende des Randoms wieder false wird, sollte der Timer weiterhin im Hintergrund verbleiben, da das Gerät nicht verfahren wurde. Doppelte Ausführung wird bisher nicht verhindert.

Aber eine Umsetzung bzw Lösung ist gar kein Problem. Die grundsätzliche Frage lautet nun:
Wann müsste der Hintergrund-Timer gelöscht werden, wenn der Random beginnt oder wenn das Gerät tatsächlich angesteuert wird? Dass keine doppelten Random Countdowns gestartet werden, stellt kein Problem dar. Ich würde bevorzugen, den Hintergrund Timer weiterhin erst zu löschen, wenn tatsächlich das Gerät gesteuert wird.

Deine flackernde Bedingung kann aber weiterhin ein Problem darstellen: wenn die Bedingung true ist und kurz vor Ablauf Random false wird (und bleibt), dann wird das Gerät nicht angesteuert. Dieses Verhalten kann ich auf Skript-Seite nicht lösen, da als Bedingung auch Türsensoren genutzt werden können.

Bzgl. des Helligkeitssensors kann ich dir empfehlen eine Hystere zu nutzen und den Helligkeitssensor virtuell neu anzulegen und in Skripten diesen zu verwenden.
Damit meine ich, dass Richtungswechsel vom Ist-Wert erst auf das virtuelle Gerät übernommen wird, wenn Differenz x überschritten wird.
Könnte in JS ziemlich fix umgesetzt werden.

gender

@GiuseppeS said in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

Diesen Fall habe ich tatsächlich nicht bedacht. Dass die Bedingung wahr wird, dann falsch und bevor der Random durch ist, wieder wahr.

Hi und danke für die schnelle Rückmeldung.

Also meine Bedingung (der Helligkeitssensor) flackert nicht. Ich benutzte einen Average-Wert (also eine Zahl), die abends wirklich kontinuierlich runtergeht.
Ich habe die Bedinung auf < 5 gesetzt und auch gestern, als ein weiterer Random-Timer angesetzt wurde (obwohl es schon einen gab), hatte sich an der erfüllten Bedingung nichts geändert bzw. der Helligkeitswert hatte sich natürlich geändert, ist einfach nur kleiner geworden (und hatte damit die Bedingung nach wie vor erfüllt).

Daher war ja meine Vermutung, dass du die Berechnung bzw. das Setzen des (nachträglichen) Random-Timers in deinem Skript grundsätzlich bei Änderung des Wertes (und nicht nach erfüllt/nicht erfüllt), der für die eingetragene Bedingung steht, durchführen lässt.

GiuseppeS

@gender
Ahhh, jetzt kommt die Erleuchtung. Ich ging von einem wechselnden Wert und somit wechselndem Status der Bedingung aus, aber (du hast Recht) tatsächlich wird auch bei weiter fallendem Wert erneut getriggert.

Werde es heute im Laufe des Nachmittags oder morgen korrigieren. Es kann dann nur ein Random gestartet werden.

GiuseppeS

EDIT: Bug bemerkt... Vis hatte nicht aktualisiert...

GiuseppeS

Changelog 18.07.2020 (Skript)

Bugfixes: Falls Random-Minuten auch für gemerkte Timer genutzt wird (bgTimerWithRandom = true), wird eine doppelte Ausführung verhindert.
Der Fall "Random-Minuten für gemerkte Timer" wird nun vollumfänglich unterstützt; d.h. der Countdown wird unterbrochen im Falle von "Reset durch nachfolgenden Timer" oder falls das Ziel-Gerät anderweitig verändert wurde.

Übrigens: falls ein gemerkter Timer mit Random-Minuten nachträglich erfüllt wird und die evtl. spätere Ausführung auch mit Random erfolgt (bgTimerWithRandom = true), dann wird in der Tabelle die Bedingungsnummer vom Timer grün blinkend dargestellt, da der Random-Countdown startet. So komplex wie der letzte Satz ist mittlerweile auch der Code
Ich hoffe die Bedienung leidet nicht unter den vielen Möglichkeiten.

@gender
Bin gespannt ob nun alles funktioniert. Das ist echt krass wie viele unterschiedliche Fälle beachtet/getestet werden können. Brauche bald eine Checkliste...

gender

@GiuseppeS said in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

Bin gespannt ob nun alles funktioniert. Das ist echt krass wie viele unterschiedliche Fälle beachtet/getestet werden können. Brauche bald eine Checkliste...

Vielen Dank für deine schnelle Korrektur.
Gestern Abend hat alles funktioniert. Die Bedingung wurde nachträglich erfüllt... der Random-Timer wurde nur einmal gesetzt und die Bedingung hat grün geblinkt
Sieht also soweit gut aus.

Mir würden übrigens noch weitere Fälle einfallen (z.B. habe ich bei mir die Besonderheit, dass das große Wohnzimmerfenster von zwei Rolladen geschlossen wird und die fährt man üblicherweise zusammen runter. Folglich bräuchte man quasi eine "Gruppe", damit die beiden zur gleichen Zeit herunterfahren, wenn die Zeit zufällig sein soll), aber ich bin so wie es ist jetzt sehr zufrieden. DANKE.

GiuseppeS

@gender
Das mit dem Gruppieren versteh ich. Hab das selbe Thema mit meinem Balkon; sind auch zwei Rollläden. Das Gruppieren im Skript wäre umsetzbar, allerdings macht es außerhalb des Skripts mehr Sinn. Mit dem "Scenes Adapter" können nicht nur Szenen sondern auch Gruppen erstellt werden.
Diese neue Scene-Gruppe könnte wiederum in die Skript Aufzählung hinzugefügt werden; Skript neustarten, fertig.

Da es eine dauerhafte Gruppe ist, würde eine Umsetzung innerhalb des Skripts nur die Komplexität erhöhen, ohne Mehrwert. Schließlich besteht nie der Bedarf, diese Gruppe in VIS spontan aufzulösen.

P.S.:
Um ein Gerät aus dem Skript bzw aus der VIS zu löschen, bitte so vorgehen: in VIS alle Timer des Geräts mit Button DEL löschen. Dann Skript stoppen, Gerät aus der Aufzählung entfernen und Skript wieder starten.

Ich hatte es bisher am Balkon so hingenommen, dass beide getrennt voneinander fahren, aber werde heute Nachmittag meinen Vorschlag selbst umsetzen

smartboart

@GiuseppeS

Hi Giuseppe....
Wenn ich selbst weitere Bedingungen hinzufügen möchte, reicht es aus folgende Scriptteile einfach fortlaufend hinzuzufügen?

createState("Timer.Editor.Condition3");
createState("Timer.Editor.Cond3State");
createState("Timer.Editor.Cond3Comp");
createState("Timer.Editor.Cond3Value");
createState("Timer.Editor.Cond3Result");

EditSubsArr[4] = on({id: ["javascript." + instance + ".Timer.Editor.Cond3Comp","javascript." + instance + ".Timer.Editor.Cond3State","javascript." + instance + ".Timer.Editor.Cond3Value"], change: "any", ack: false}, function (obj) {
        var ConditionJSON = JSON.parse(getState("javascript." + instance + ".Timer.Editor.ConditionJSON").val);
        var Cond3State = getState("javascript." + instance + ".Timer.Editor.Cond3State").val
        var Cond3Comp = getState("javascript." + instance + ".Timer.Editor.Cond3Comp").val
        var Cond3Value = getState("javascript." + instance + ".Timer.Editor.Cond3Value").val
        var strCond3 = "getState(\"" + ConditionJSON[Cond3State] + "\").val " + Cond3Comp + " " + Cond3Value
        setState("javascript." + instance + ".Timer.Editor.Condition3", strCond3);
        if (Cond3State != "" && Cond3Comp != "" && Cond3Value != "") {
            setState("javascript." + instance + ".Timer.Editor.Cond3Result", eval(strCond3));
        }
    });
        case 3:
            strEval1 = eval(DeviceJSON.Conditions[1].ConditionStr);
            strEval2 = eval(DeviceJSON.Conditions[2].ConditionStr);
            strEval3 = eval(DeviceJSON.Conditions[3].ConditionStr);
            sumEval = (strEval1 && strEval2 && strEval3);
            break;


            "3":{
                "ConditionStr": "",
                "CondState": "",
                "CondComp": "==",
                "CondValue": ""
            }

GiuseppeS

@smartboart
Ich müsste mir das am PC genauer anschauen. Was definitiv fehlt ist die Auswertung der Bedingungen:

function condEval(DeviceJSON){
... 
}

Da eine zukünftige Version höhere Anzahl von Bedingungen unterstützen soll, müsste ich genau diese Funktion mit einem generischen Ansatz programmieren.
Es wären einige Neuerungen notwendig...

Vorerst kannst du es probieren, selbst die Erweiterung einzupflegen können in der Testphase auch chatten, wenns zeitlich passt.

Aktuell bin ich dabei, eine HTML Ansicht des Editors zu erstellen. Es geht voran, aber mühselig, dass es gut ausschaut
Im nächsten Schritt muss der html Code im Skript erstellt werden.
Das Projekt dauert daher noch ca. 2 Wochen.

smartboart

@GiuseppeS

danke für das Anbgebot...hab jetzt überall mal angepasst wo ich denke dass es sinn macht...
Mit Vortlaufenden Nummern macht aber im Code an den

EditSubsArr[5]

spätesten Robleme, da dise bereits vortlaufend sind...
Für ene 4 Bedingung müsste nach meiner denke dann für den EditSubsArr[] ne ander Nummer herhalten..
Worauf das wieder Auswirkungen hat, steige ich noch nicht durch....

GiuseppeS

@smartboart

Das kannst du ruhig machen mit der fortlaufenden Nummer bei EditSubsArr[]
Nimm die nächste freie Ziffer. Die letzte müsstest du ganz unten finden, in dieser Reihe von Subscribtions. Beim Löschen dieser Subscribtions wird die Länge ermittelt, bedeutet, dass deins auch gelöscht wird.

Hintergrund dieser optionalen Subsribtions:
Bei der Verwendung von nur einem Editor-PopUp musst ich steuern welches Skript auf die Editor-Eingaben reagieren soll. Diese Subscribtions werden aktiviert, sobald ein Click in Tabelle erfolgt. Wird auch im Log ausgegeben, wenn Debug im Skript aktiviert wird.

smartboart

@GiuseppeS sagte in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

EditSubsArr[]

ok, verstanden...

        //for (let i = 1; i <= 3; i++){
        //selbst geändert
        for (let i = 1; i <= 4; i++){

habe alle for Schleifen auf 4 geändert...
Jetzt ist beim ofnnen der Konfiguration folgendes Fehlerbild...

javascript.1	2020-07-26 16:33:06.122	error	(1703) at processImmediate (internal/timers.js:456:21)
javascript.1	2020-07-26 16:33:06.122	error	(1703) at Immediate._onImmediate (/opt/iobroker/node_modules/iobroker.js-controller/lib/adapter.js:5384:37)
javascript.1	2020-07-26 16:33:06.121	error	(1703) at Object.stateChange (/opt/iobroker/node_modules/iobroker.javascript/main.js:451:25)
javascript.1	2020-07-26 16:33:06.121	error	(1703) at Object.callback (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1052:38)
javascript.1	2020-07-26 16:33:06.120	error	(1703) at Object.<anonymous> (script.js.common.Zeitsteuerung.Variable_Zeitsteuerung_Vis:804:17)
javascript.1	2020-07-26 16:33:06.120	error	(1703) at openEditor (script.js.common.Zeitsteuerung.Variable_Zeitsteuerung_Vis:711:112)
javascript.1	2020-07-26 16:33:06.119	error	(1703) Error in callback: TypeError: Cannot read property 'ConditionStr' of undefined

GiuseppeS

@smartboart sagte in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

ConditionStr

Das JSON, welches alle Timer beinhaltet muss natürlich auch aufgebohrt werden.
Unter dem Key Conditions sind nur 3 Objekte angesiedelt, hier müsste das 4. noch ergänzt werden. Könntest du erstmal mit einem Online-JSON-Editor bewerkstelligen; ansonsten gehts natürlich auch im Code.
So schaut es aktuell aus:
Es ist nur Vorhalt für 3 Timer vorhanden, denke daher kommt dein Fehler...

smartboart

@GiuseppeS sagte in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

JSON

ja tatsächlich... Geräte welche ich später in die Enumeration DeviceEnum aufgenommen haben, haben nun bereits meine 4 conditions hinterlegt...
Bzw Geräte welche schon hinterlegt waren aber nocht nicht programmiert..

Geräte welche bereits eine Zeitsteuerung im editor erfahren haben, haben meine 4. Bedingung nocht nicht.. Sollte so also funktionieren...

Müsste halt die json löschen und alles neu programmieren....oder wie due sagst im editor ändern...

GiuseppeS

@smartboart

Hängt nun davon ab, wie komplex deine bisherigen Timer waren... Komplett bei null anfangen könnte aufwendiger sein, als diesen weiteren Key zu ergänzen. Ich würde es deine Stelle zumindest nicht programmatisch lösen.

smartboart

@GiuseppeS
ja

@GiuseppeS sagte in [Vorlage] Variable Zeitsteuerung mit VIS Editor:

Hängt nun davon ab, wie komplex deine bisherigen Timer waren... Komplett bei null anfangen könnte aufwendiger sein, als diesen weiteren Key zu ergänzen. Ich würde es deine Stelle zumindest nicht programmatisch lösen.

ich war noch am Testen, so viele habe ich noch nicht programmiert...
Werde das jetzt erstmal testen ob meine 4. Bedingung tut, dann packe ich noch 2 drauf ...Dann werde ich die Timer neu programmieren...

Vielen Dank für die Unterstützung...

GiuseppeS

Habe nun einen ersten (zufriedenstellenden) Entwurf des Editors in HTML.

Aus dem Code werden folgende Optionen der Darstellung herausgelöst:

Schriftfarbe bei Texten und Button/Select (hier: weiß)
Rahmen der Button/Select (hier: weiß); Grün/Rot sind "funktional", daher nicht einfach editierbar
Füllfarbe der Button/Select (hier: schwarz); es funktioniert übrigens auch "transparent" oder rgba mit Alphawerten

Bedingungen werden über Plus-Button ergänzt und mit Del-Button gelöscht. Klasse ist, dass bei manueller Zeitauswahl die Funktionalität des Browsers genutzt wird, d.h. bei Android-Geräten schaut das z.B. so aus:
Screenshot_20200726_173135_com.android.chrome.jpg

GiuseppeS

So würde es mit "rgba(255,255,255,0.2)" ausschauen, statt "#fff":

Wenn keine Gegen- bzw. Verbesserungsvorschläge eingehen, würde ich damit weitermachen.

EDIT:
Astro1 und Astro2 sind zwei alternative Darstellungen. Im Fall1 wird "manuell" ausgewählt, d.h. rechts erscheint ein Zeiteingabefeld. Im Fall2 wird eine Astrozeit gewählt, d.h. rechts erscheint eine Möglichkeit für die Offset-Korrektur.

NEWS

[Vorlage] Variable Zeitsteuerung mit VIS Editor

Support us

1.2k
Online

30.6k
Users

77.2k
Topics

1.2m
Posts

NEWS

[Vorlage] Variable Zeitsteuerung mit VIS Editor

Support us

1.2kOnline

30.6kUsers

77.2kTopics

1.2mPosts

1.2k
Online

30.6k
Users

77.2k
Topics

1.2m
Posts