[Vorlage] Variable Zeitsteuerung mit VIS Editor

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.

sigi234

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

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

Ist der Hintergrund auswählbar? Kann man umschalten Helles/Dunkles Design?

Glasfaser

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

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

Ist halt Geschmacksache .... ist ein wenig aufgebläht ..

finde es immer noch so besser ... strukturierter Aufbau .

Glasfaser

@GiuseppeS

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

GiuseppeS

@sigi234
Meinst du den Hintergrund der Buttons? Der blaue Hintergrund ist von mir.

sigi234

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

Der blaue Hintergrund ist von mir.

Den blauen

GiuseppeS

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

@GiuseppeS

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

Scherzkeks

Was den Punkt bzgl. Optik betrifft: Bin selbst auch hin- und hergerissen.
In HTML wären mehrere Vorteile:

Darstellung ist dynamisch, d.h. es werden keine Elemente aktiviert oder deaktiviert, sie werden so erstellt wie benötigt. Das macht sich in der Höhe bemerkbar; so gering wie möglich.
Bedingungen sind dynamischer zu ergänzen/löschen.
In Bedingungen könnte ich als Zielwert dynamisch ein Select-true/false oder ein freies Input-Feld erzeugen. Aktuell kaum möglich...
Es wird nur noch ein View benötigt.

Die HTML Ergänzung würde übrigens auf die aktuelle PopUp-Editor-Bedienung aufsetzen. Der alte Editor könnte wahrscheinlich weiter genutzt werden.

Allerdings würde ich die HTML-Ergänzung ehrlich gesagt nicht weiter verfolgen, wenn es allgemein nicht akzeptiert wird. Habe bisher nicht viel Arbeit rein gesteckt. Das waren heute ca. 3 Stunden. Der programmatische Anteil wäre deutlich aufwendiger.

NEWS

[Vorlage] Variable Zeitsteuerung mit VIS Editor

Support us

1.2k
Online

30.2k
Users

76.3k
Topics

1.2m
Posts

NEWS

[Vorlage] Variable Zeitsteuerung mit VIS Editor

Support us

1.2kOnline

30.2kUsers

76.3kTopics

1.2mPosts

1.2k
Online

30.2k
Users

76.3k
Topics

1.2m
Posts