[gelöst] on{... , change: "any"} läuft immer 2x mal

[mickym]

Doch meines Erachtens ist das genau so - ich habe das mit Node-Red nachvollzogen.
Bau einfach "Bestätigt" = Yes ein.

[Dado]

Habe das mal geändert, aber das scheint immer noch dasselbe zu sein...

on({id: sourceIDs , change: "any", ack: true}, function (obj) {
   log("Old State: "  + JSON.stringify(obj.oldState.val), "warn");
   log("ack: "  + JSON.stringify(obj.newState.ack), "warn");
//    light_control(obj.id);
});

4.5.2020, 15:10:54.444	[info ]: javascript.0 (689) Stop script script.js.common.Light.motion-light1
4.5.2020, 15:10:54.572	[info ]: javascript.0 (689) Start javascript script.js.common.Light.motion-light1
4.5.2020, 15:10:54.587	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: Subcribe folgende source IDs: ["deconz.0.Sensors.24.presence","deconz.0.Sensors.3.open"]
4.5.2020, 15:10:54.588	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: subscribe: {"pattern":{"id":"deconz.0.Sensors.24.presence","change":"any","ack":true,"q":0},"name":"script.js.common.Light.motion-light1"}
4.5.2020, 15:10:54.588	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: subscribe: {"pattern":{"id":"deconz.0.Sensors.3.open","change":"any","ack":true,"q":0},"name":"script.js.common.Light.motion-light1"}
4.5.2020, 15:10:54.588	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: registered 2 subscriptions and 0 schedules
4.5.2020, 15:11:12.577	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: Old State: false
4.5.2020, 15:11:12.577	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: ack: true
4.5.2020, 15:11:12.593	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: Old State: true
4.5.2020, 15:11:12.593	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: ack: true

[mickym]

@Dado finde ich nicht. Ist zwar beides Mal ack=true - aber einmal ist doch der Value FALSE und einmal TRUE

[paul53]

@mickym sagte:

aber einmal ist doch der Value FALSE und einmal TRUE

Nein, das ist der oldState.val. Interessant wäre gleichzeitig der state.val.

[Dado]

Da kann ich helfen
Das hier ...

on({id: sourceIDs , change: "any", ack: true}, function (obj) {
   log("oldState.val: "  + JSON.stringify(obj.oldState.val), "warn");
   log("newState.ack: "  + JSON.stringify(obj.newState.ack), "warn");
   log("state.val: "  + JSON.stringify(obj.state.val), "warn");
//    light_control(obj.id);
});

...ergibt bei einmal durch den Sensor laufen

4.5.2020, 15:23:23.868	[info ]: javascript.0 (689) Stop script script.js.common.Light.motion-light1
4.5.2020, 15:23:23.958	[info ]: javascript.0 (689) Start javascript script.js.common.Light.motion-light1
4.5.2020, 15:23:23.962	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: Subcribe folgende source IDs: ["deconz.0.Sensors.24.presence","deconz.0.Sensors.3.open"]
4.5.2020, 15:23:23.962	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: subscribe: {"pattern":{"id":"deconz.0.Sensors.24.presence","change":"any","ack":true,"q":0},"name":"script.js.common.Light.motion-light1"}
4.5.2020, 15:23:23.962	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: subscribe: {"pattern":{"id":"deconz.0.Sensors.3.open","change":"any","ack":true,"q":0},"name":"script.js.common.Light.motion-light1"}
4.5.2020, 15:23:23.963	[info ]: javascript.0 (689) script.js.common.Light.motion-light1: registered 2 subscriptions and 0 schedules
4.5.2020, 15:23:34.165	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: oldState.val: false
4.5.2020, 15:23:34.165	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: newState.ack: true
4.5.2020, 15:23:34.166	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: state.val: true
4.5.2020, 15:23:34.168	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: oldState.val: true
4.5.2020, 15:23:34.171	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: newState.ack: true
4.5.2020, 15:23:34.172	[warn ]: javascript.0 (689) script.js.common.Light.motion-light1: state.val: true

[J.A.R.V.I.S.]

@Dado du könntest 'change' in 'ne' ändern, dann wird er nur getriggert, wenn der Wert ungleich des alten Wertes ist.

Bleibt der Wert eine gewisse Zeit auf true oder geht er danach wieder auf false?

[paul53]

@Dado
Offenbar ist es eine Eigenheit des Deconz-Adapters, dass unmittelbar nach einer Wertänderung nochmal aktualisiert wird. Triggern auf change: "ne" sollte das Problem beheben.

[Dado]

@J-A-R-V-I-S
Der Wert bleibt auf true stehen und geht nach 90 Sekunden wieder auf false...
Was etwas seltsam ist, da ist das Teil so modifiziert habe, dass er nur noch eine Totzeit von 5 Sekunden hat (Standard sind 2 Min)

[mickym]

@Dado Ich habe die Dinger auch. Meines Erachtens sind das 2 verschiedene Paar Stiefel - das Löten heißt nur, dass er den Status mit einem minimalen Zeitpunkt von 5 Sek. auflöst.
Die 90s kommen, wenn Du in occupancy_timeout nichts eingetragen hast. Trag da einfach mal einen Wert ein, aber ich würde den nicht zu klein wählen

[Dado]

@paul53
Ja und nein...
Ja, ich bekomme nur noch einmal getriggert
Nein, jetzt triggert er nur noch bei Flankenwechsel (was bei != ja auch erwartbar ist)

Das ist bei einem Bewegungsmelder aber ein "schwieriges" Verhalten oder anders ausgedrückt, der WAF ist dann gleich Null

[paul53]

@Dado sagte:

Das ist bei einem Bewegungsmelder aber ein "schwieriges" Verhalten

Dann verwende "entprellen".

var timer;
on({id: sourceIDs}, function(dp) {
    if(!timer) {
        timer = setTimeout(function() {
            timer = null;
        }, 100);
        // Aktionen
    }
});

[J.A.R.V.I.S.]

@Dado die Frage ist, möchtest du, wenn du in der Zeit noch einmal den Bewegungsmelder triggerst, das Skript nochmal ausführen? Das wäre mit "ne" dann nicht möglich. Dann müsstest du "any" und dann den Timestamp prüfen, ob z.B. größer eine Sekunde zum letzten mal.

[Dado]

@paul53
Stand da nicht gerade noch was mit Suche im Forum?
Ja, sowas hatte ich gerade im Forum gefunden. Aber danke nochmal fürs Rein'paste'n!
Das hilft mir weiter. Werde beizeiten noch mal Rückmeldung geben, ob das dann wie gewüscht funktioniert

Danke!

[Dado]

@J-A-R-V-I-S
Brauche zweiteres. Das Licht soll ja länger anbleiben, wenn man im Sensorbereich steht. Werde diese Idee mal mit dem Snippet von paul angehen.

Danke!

[Dado]

Sieht gut aus! Eine (eventuell dumme) Frage hätte ich noch:
Das Skript soll ja auf verschiedene Sensoren reagieren. Wenn jetzt 2 verschiedene Sensoren innerhalb des Timeout Fensters triggern, dann würde der 2. nicht ausgeführt, oder?

Macht es Sinn, dass dann ggf. so zu modifizieren (und timer als array zu initialisieren)?

    if(!timer[obj.id]) {
        timer[obj.id] = setTimeout(function() {timer[obj.id] = null;}, 100);
//      light_control(obj.id);
    }

[paul53]

@Dado sagte:

Wenn jetzt 2 verschiedene Sensoren innerhalb des Timeout Fensters triggern, dann würde der 2. nicht ausgeführt, oder?

Richtig. Wie wahrscheinlich ist es ? Man kann sicherlich auch die Sperrzeit verringern (z.B. 20 ms) , um die Wahrscheinlichkeit weiter zu verringern.

[Dado]

@paul53
Vermutlich weit weniger wahrscheinlich, als ich denke... Ist ja jetzt nicht so, dass ich damit ein komplettes Hochhaus steuere.
Danke für den Realitätscheck.
Werde ggf mal den Timeout zart reduzieren und stelle das hier mal auf "gelöst"

[ticaki]

Würde gerne noch etwas einbringen. Soweit ich sehe verwendest du obj.state.val nicht. Dann könntest du val:true benutzen.

Ob das mit dem timer[obj.id] Sinn macht. Ja ist doch kaum Arbeit. Ich würde es dann aber eher so lösen:

var devices = {};
on(...(obj)=> {
 devices[obj.id] = true;
 //hier entprellen
 setTimeout(meineFunktion,100);
})

function meineFunktion() {
 for (let dev in devices){...}
 devices = {};
}

Kann Syntaxfehler enthalten.