[gelöst] Fensterkontakt steuert Thermostat - Logikfehler?

[Slowman]

@paul53

Danke, werde jetzt mal den Blockly für mich anpassen und dann mal zur Kontrolle posten.
Der 2. kleine Blockly ist ein extra Blockly oder mit rein-

[paul53]

@Slowman sagte:

oder mit rein-

Der zweite Trigger muss in das gleiche Skript, da die Übergabe der Heizperiode an die erste Trigger-Funktion über eine Variable erfolgt.

Wert findet man unter "Trigger".

[Slowman]

@paul53

So mal nach gebaut, aber so richtig sehe ich noch nicht durch. So richtig? Wie mache ich das mit der Heizperiode an/aus. Müsste ja ein extra Script sein.

Aber glaube nicht mehr ganz so blind. Zu der Heizung noch ne Erklärung ist ein Tado, dort habe ich beim alten Script, zu erst Power eingeschaltet bzw. aus und Automatik springt ins Programm zurück.

[paul53]

@Slowman sagte:

Zu der Heizung noch ne Erklärung ist ein Tado

Kenne ich nur vom "hören sagen".

@Slowman sagte in Fensterkontakt steuert Thermostat - Logikfehler?:

Wie mache ich das mit der Heizperiode an/aus.

Ein eigener Datenpunkt (unter "0_userdata.0"), der z.B. per Vis geschaltet werden kann.

@Slowman sagte in Fensterkontakt steuert Thermostat - Logikfehler?:

So richtig?

Der Vergleich muss mit dem Wert des Datenpunktes "Power" (Heizung on) erfolgen.

[Slowman]

@paul53

Schaue bitte noch mal rüber:

Wie gesagt ich schalte erst das Thermostat an und danach auf Automatikprogramm von Tado.

Sollte der Script so jetzt laufen?

[paul53]

@Slowman sagte:

Sollte der Script so jetzt laufen?

Der Trigger für die Heizperiode verwendet sicherlich einen falschen Datenpunkt. Ansonsten sieht es aus Blockly-Ansicht plausibel aus. Um es genau beurteilen zu können, poste bitte den erzeugten Javascript-Code ohne die letzte Zeile in Code tags.

[Slowman]

@paul53

Den Datenpunkt mit true (aktiv) erstellt, aber wie greift der Script darauf zu?

und die Javaausgabe :

var Heizperiode, Heizung, Meldung;


// Fensterkontakt
on({id: 'zigbee.0.00158d0004524503.opened', change: "ne"}, function (obj) {
  var value = obj.state.val;
  var oldValue = obj.oldState.val;
  if (Heizperiode) {
    if ((obj.state ? obj.state.val : "")) {
      // Variablen
      Heizung = 'OFF';
      Meldung = 'Fenster offen';
    } else {
      Heizung = 'ON';
      Meldung = 'Fenster geschlossen';
      // Heizung wird auf Automatik gestellt
      setStateDelayed("tado.0.125400.Rooms.8.overlay.clearZoneOverlay"/*Clear Zone Overlay*/, true, 120000, false);
    }
    if (getState("tado.0.125400.Rooms.8.setting.power").val != Heizung) {
      // Heizung on
      setStateDelayed("tado.0.125400.Rooms.8.setting.power"/*Power*/, Heizung, 60000, false);
    }
    sendTo("telegram", "send", {
        text: Meldung
    });
  }
});

on({id: 'tado.0.125400.Rooms.8.setting.power', change: "ne"}, function (obj) {
  var value = obj.state.val;
  var oldValue = obj.oldState.val;
  Heizperiode = (obj.state ? obj.state.val : "");
  if (!Heizperiode) {
    // Heizung on
    setState("tado.0.125400.Rooms.8.setting.power"/*Power*/, 'OFF');
  }
});

// Heizung on off Tado Datenpunkt

[paul53]

@Slowman sagte:

on({id: 'tado.0.125400.Rooms.8.setting.power', change: "ne"}, function (obj) {

Wie schon gesagt: Der Datenpunkt taugt nicht für die Heizperiode. Der Rest passt.
Bei den Verzögerungen sollte man den Haken bei "löschen falls läuft" setzen, da sonst mehrere Timer laufen können.

@Slowman sagte in Fensterkontakt steuert Thermostat - Logikfehler?:

aber wie greift der Script darauf zu?

Den Datenpunkt dem zweiten Trigger per Select ID zuweisen.

[Slowman]

@paul53

So, denke mal da sich es verstanden habe, was du geschrieben hast. Dürfte jetzt korrekt sein.

var Heizperiode, Heizung, Meldung;


// Fensterkontakt
on({id: 'zigbee.0.00158d0004524503.opened', change: "ne"}, function (obj) {
  var value = obj.state.val;
  var oldValue = obj.oldState.val;
  if (Heizperiode) {
    if ((obj.state ? obj.state.val : "")) {
      // Variablen
      Heizung = 'OFF';
      Meldung = 'Fenster offen';
    } else {
      Heizung = 'ON';
      Meldung = 'Fenster geschlossen';
      // Heizung wird auf Automatik gestellt
      setStateDelayed("tado.0.125400.Rooms.8.overlay.clearZoneOverlay"/*Clear Zone Overlay*/, true, 120000, true);
    }
    if (getState("tado.0.125400.Rooms.8.setting.power").val != Heizung) {
      // Heizung on
      setStateDelayed("tado.0.125400.Rooms.8.setting.power"/*Power*/, Heizung, 60000, true);
    }
    sendTo("telegram", "send", {
        text: Meldung
    });
  }
});

on({id: 'javascript.0.Eigene_Datenpunkte.Heizung.Heizperiode', change: "ne"}, function (obj) {
  var value = obj.state.val;
  var oldValue = obj.oldState.val;
  Heizperiode = (obj.state ? obj.state.val : "");
  if (!Heizperiode) {
    // Heizung on
    setState("tado.0.125400.Rooms.8.setting.power"/*Power*/, 'OFF');
  }
});

// Heiperiode Datenpunkt gesetzt

Der letzte Blockly ist dafür da, das wenn DP Heizperiode false ist, der obrige Blockly gar nicht erst gestartet wird, richtig?

[paul53]

@Slowman sagte:

Dürfte jetzt korrekt sein.

Sieht so aus.

[Slowman]

@paul53

Keine Auschaltung, keine Meldung per Telegram. Kann der Fehler vielelicht daran liegen das ich einmal auf Automatikprogramm Trigger und ein mal auf Power? Den Trigger auf das Automatikprogramm brauche ich auch.

[paul53]

@Slowman sagte:

Keine Auschaltung, keine Meldung per Telegram.

Initialisiere die Variable Heizperiode bei Skript-Start.

[Slowman]

@paul53

Funktioniert jetzt, vielen Dank für die Hilfe und natürlich die Geduld

Für mich zum Verständnis, man sollte nicht so viele Datenpunkte anlegen (was ich gemacht habe), sondern lieber mit Variablen arbeiten, damit man das ioBroker System nicht so belastet? Wenn ich den Script jetzt mir anschaue, habe ich nur die "Hardware" DP abgefasst und nur einen DP angelegt.

[paul53]

@Slowman sagte:

sondern lieber mit Variablen arbeiten, damit man das ioBroker System nicht so belastet?

Ja.

[Slowman]

@paul53

Kleines kosmetisches Problem noch, wenn ich mehrere Fenster in einem Raum öffne bekomme ich für jedes geöffnete Fenster im Raum eine Benachrichtigung.

[amg_666]

@Slowman Das ist aber nur bedingt ein "Problem". Wenn du das nicht willst müsstest du dir noch ne Variable "fenster_auf" spendieren und die auf true setzen sobald ein Fenster geöffnet ist. und dann darfst du die Benachrichtigung nur bei "fenster_auf"=True senden.
Ich hab das bei mir nicht gemacht, weil Fenster auf zum einen die Heizung runterfährt, bei mir aber auch die Alarmanlage auslöst wenn niemand im Haus ist. Und da will ich für jedes Fenster wissen ob es geöffnet wurde, egal ob schon eins auf ist.

[paul53]

@Slowman sagte:

bekomme ich für jedes geöffnete Fenster im Raum eine Benachrichtigung.

Wenn das nicht gewollt ist, schiebe Telegram in in den falls-Block "Wert von Power != Heizung".

[Slowman]

@paul53

Habe jetzt einiges rumprobiert und klappt nicht. Also gelesen ja, nur die Umsetzung des Typen für dem Bildschirm hapert es.

In diesen Block kommen die

Funzt nicht, denke mal falsche Stelle und die Meldung Fenster im Raum geschlossen auch erst, wenn alle Fenster zu sind sind.

[paul53]

@Slowman sagte:

klappt nicht.

Was klappt nicht ? Bitte das ganze Blockly zeigen.

@Slowman sagte in [gelöst] Fensterkontakt steuert Thermostat - Logikfehler?:

Meldung Fenster im Raum geschlossen auch erst, wenn alle Fenster zu sind sind.

So soll es sein ?

[Slowman]

@paul53

Also, es wird immer noh je geöffneten Fenster eine Meldung per Telegram ausgegeben und auch beim Schliessen von jedem Fenster. Es sollte so sein, das wenn 1 Fenster geöffnet ist und das 2 dazu geöffnet wird nur eine Meldung kommt, auch beim Schliessen eine Meldung kommt, wenn das letzte Fenster geschlossen wird. Die Heizung startet ja lt. Script sowieso erst, wenn alle Fenster geschlossen sind und schaltet sich aus, wenn 1 Fenster geöffnet ist.