Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DelayedExecutionCondition #32

Closed
bgewehr opened this issue Mar 22, 2016 · 17 comments
Closed

DelayedExecutionCondition #32

bgewehr opened this issue Mar 22, 2016 · 17 comments

Comments

@bgewehr
Copy link
Contributor

bgewehr commented Mar 22, 2016

Michael, erst mal dank für die Conditions!

Wenn Du noch List hast, können wir nun das voraussichtlich letzte Thema anzugehen, die sog. DelayedExecutionCondition des Weekdaytimers in fhem.

Damit ist folgendes möglich:
"Am Montag um 22:00 schließe den Rolladen, allerdings nur dann, wenn die Terrassentür nicht geöffnet ist. Falls die Tür geöffnet ist, warte mit der Ausführung, bis sie geschlossen wird, dann schließe den Rolladen."

Oder:
"Werktags um 16:00 Uhr stelle die Heizkörper im Bad auf 21 Grad, allerdings nur dann, wenn wir zuhause sind (Condition) und wenn das Badezimmerfenster nicht geöffnet ist. Wenn das Fenster geöffnet ist, warte bis es geschlossen wurde, dann stelle die Temperatur ein. (DelayedExecutionCondition)"

Technisch ist das eine zweite Condition derselben Art wie die erste, aber mit der Bezeichnung DelayedExecutionCondition. Sonst alles wie gehabt.

Details:
http://fhem.de/commandref.html#WeekdayTimer

Hast Du noch Lust oder machen wir erst mal eine Pause?

@mworion
Copy link
Owner

mworion commented Mar 23, 2016

Hallo Bernd,

können das fertigmachen, sofern der erst schon läuft. Nur für mich zum verstehen: Im Grunde geht es darum eine weitere Zeile reinzubringen, wo eine andere Art von Conditions eingegeben werden kann ?
Langsam wird das aber richtig voll!

@mworion
Copy link
Owner

mworion commented Mar 23, 2016

Habe mir das in der FHEM Spek angesehen. Könnte auch mit einem weiteren Feld bei der Conditions Zeile gehen. Was ich brauche ist die Liste der Parameter, die Du eingeben / verändern willst.

@bgewehr
Copy link
Contributor Author

bgewehr commented Mar 23, 2016

Es handelt sich bei der DelayedExecutionCondition um dieselbe Machart wie die Condition. Vergleich Device Value oder Perl String. Also "einfach" das Gleiche noch einmal.

Btw: Ich habe heute erfolgreich das Anheben der Schlafzimmer Rolläden durch eine Condition (NRW_Ferientag ne "1") verhindert. Ein Traum...

@mworion
Copy link
Owner

mworion commented Mar 23, 2016

Hallo Bernd,

probier die v4.4 aus, sollte alles drin sein.

Aber Achtung !!!!! :

Ich musste der Ordnung wegen die key/value pairs im JSON nochmal ändern. Da ich die FHEM Zeilen jetzt doppelt habe, ist die Beschreibung 'condition' aus dem key bei 'conditionType', usw. herausgefallen. Es ist jetzt nur nor 'type'. Die Zuordnung wird ja schon im übergeordneten Struct getroffen. Die Beschrebung war ja doppelt drin und die Zeilen sind kaum noch lesbar gewesen. Bitte korrigiere das in Deiner Auswertung. Danke Dir !

Frohe Ostern wennes so passen würde !

Michel

@bgewehr
Copy link
Contributor Author

bgewehr commented Mar 23, 2016

Das Osterfest ist gerettet! Dank Dir! Ich berichte dann!

@mworion
Copy link
Owner

mworion commented Mar 23, 2016

Da freue ich mich ja 😀.
Gestaltungs und Textwünsche natürlich offen.

@mworion
Copy link
Owner

mworion commented Mar 26, 2016

Bernd,

bei mir aktuell die v4.5. Funktional nichts geändert. Sie etwas besser aus, habe den Design Type = 1 entfallen lassen, weil ohnehin keine Nutzer da. FHEM bleibt aber weiterhin designType = 2

@bgewehr
Copy link
Contributor Author

bgewehr commented Mar 26, 2016

Großartig! Die fhem community arbeitet schon am neuen uzsu_execute... Tolle Sache! Ich schau es mir an.

@bgewehr
Copy link
Contributor Author

bgewehr commented Mar 26, 2016

4.5 gefällt mir sehr gut! Due vertikalen Texte passen super und erklären alles.

Btw: Hast Du den act button für sunrise noch auf dem Schirm?

@mworion
Copy link
Owner

mworion commented Mar 27, 2016

Ja, habe ich. Muss mir aber noch Gedanken machen, dass in die gleiche Logik umsetze.

@mworion
Copy link
Owner

mworion commented Mar 28, 2016

Hallo Bernd,

Sunrise und Sunset ist umgestellt.
Die Formatrierung (Act Button rechtsbündig will ich noch machen)
Ich habe noch einen Darstellungsfehler beim Erstellen einer Zeile

Wichtig: Ich habe nochmals den JSON anpassen müssen: Nur eine Bezeichnungsänderung. Ich habe dort wo 'Perl' steht das gegen 'String' geändert. Warum: Wenn ich die Funktionen in smarthome nachbaue, dann kann keiner etwas mit 'Perl' anfangen, weil dort alles in Python ist. Ist im Header auch beschrieben. Sorry dafür, muss ich aber vor dem Release machen. Später will dort keine mehr anpacken.

@mworion
Copy link
Owner

mworion commented Mar 28, 2016

Ach ja: ist die V4.6

@bgewehr
Copy link
Contributor Author

bgewehr commented Mar 28, 2016

Super! Kein Ding, das ist schnell umgebaut.
Ich bin wegen Smartphone Display immer noch auf der Such nach Platz für die WE Buttons. Neben Sunrise wäre dich auch jetzt noch noch Platz im Displaytype 2?

@mworion
Copy link
Owner

mworion commented Mar 28, 2016

Hallo Bernd,

mein Fehler habe ich gefunden und gefixed.
Die WE udn !WE habe ich in die zeile SUN reingepackt
Alle Felder, die nicht mit Act aktiuviert wurden, können nicht mehr beschrieben werden.

Gib mal Feedback, was Du davon hälst

@mworion
Copy link
Owner

mworion commented Mar 28, 2016

So mit der v4.8 sollte mal alles (bis auf die JSON umstellung alles mal da sein. jetzt ist noch Fehlersuche angesagt. Komme aber heute nicht mehr weiter.

@mworion
Copy link
Owner

mworion commented Apr 2, 2016

Wie sieht es mit diesem Feature im Test aus ?

@bgewehr
Copy link
Contributor Author

bgewehr commented Apr 8, 2016

Für mich ist das hier erfolgreich abgeschlossen!

@bgewehr bgewehr closed this as completed Apr 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants