[gelöst] Javascript-adapter ruft on ereignis doppelt auf
-
@haus-automatisierung sorry.
Es wird ein Shelly 2.5 geschaltet.
Diesen schalte ich mit setstatedelayed.
Jeweils(val=true oder false) mit dem ack nicht angegeben, also false.
Die on Methode, welche ich auf ack=true Abfrage wird zwei mal mit val=true und ack=true aufgerufen. Aber frage ich auf lediglich die id ab, dann wird bei Änderung nur einmal mit ack = fAlse aufgerufenMein Gedanke war nun, dass irgendwie bei der Abfrage auf ack=true der Wert für ack nicht richtig übergeben wird.
-
@ben1983 sagte in Javascript-adapter ruft on ereignis doppelt auf:
Mein Gedanke war nun, dass irgendwie bei der Abfrage auf ack=true der Wert für ack nicht richtig übergeben wird.
Müsste ich in Ruhe testen, aber würde mich sehr wundern wenn das passiert. Die Versionen werden ja von tausenden Leuten schon genutzt. Das wäre sicherlich aufgefallen
-
@haus-automatisierung ja,
Das hat ja auf meiner Windows Installation auch funktioniert.
Jetzt bei der neuen auf Docker trat es auf. -
@paul53 @haus-automatisierung Das hier habe ich wie beschrieben eingefügt:
on({id: IdFrischwasserstellmotor}, (dp) => { log('Frischwasser: ' + dp.state.val + ', Ack: ' + dp.state.ack + ', From: ' + dp.state.from); });Im Log steht dann:
16:27:46.649 info javascript.0 (32410) script.js.common.eventtest: Frischwasser: false, Ack: false, From: system.adapter.javascript.0 16:27:46.652 info javascript.0 (32410) script.js.common.eventtest: Frischwasser: false, Ack: true, From: system.adapter.shelly.0 16:27:46.930 info javascript.0 (32410) script.js.common.eventtest: Frischwasser: false, Ack: true, From: system.adapter.shelly.0also für michsieht es so aus, dassichim script auf false setze und der shelly adapter 2 x mit true zurück meldet.
Oder? -
@ben1983 sagte: im script auf false setze und der shelly adapter 2 x mit true zurück meldet. Oder?
Ja, so sieht es aus: Im Abstand von ca. 300 ms wird val = false und ack = true gesendet.
-
@ben1983 sagte in Javascript-adapter ruft on ereignis doppelt auf:
also für michsieht es so aus, dassichim script auf false setze und der shelly adapter 2 x mit true zurück meldet.
Ich würde das ja gerne nachvollziehen oder testen. Aber Du lieferst ja keine Infos
- Shelly 2.5 in welchem Modus (Switch oder Shutter)
- Welche Firmware-Version
- Welcher Datenpunkt genau
- Wie integriert (MQTT oder CoAP?), ...
-
@haus-automatisierung wie es ausschaut sind es alle shellys (shelly 1,shelly2,shelly2.5) die ich im Einsatz habe.
Alle über CoAP.
Alle auf der neuesten Firmware: Datenpunkt "Relais0.switch" "Relais1".switch (Wobei ich andere jetzt nicht abfrage)
Installiert ist die Beta:
-
@ben1983 sagte in Javascript-adapter ruft on ereignis doppelt auf:
Alle auf der neuesten Firmware: Datenpunkt "Relais0.switch" "Relais1".switch (Wobei ich andere jetzt nicht abfrage)
Fehler gefunden. Problem war, dass die Änderung einmal bestätigt wurde, wenn der Wert geändert wurde und zum Gerät geschickt wird. Zusätzlich antwortet das Gerät dann aber direkt über CoAP oder MQTT direkt mit dem neuen Wert. Was wiederum zu einem Event mit ack geführt hat.
Ist in 5.3.1 behoben.
-
@haus-automatisierung ok, danke
-
@haus-automatisierung wann wird die Version im stable sein?
-
@ben1983 Wenn Du den Support übernimmst, mache ich das gleich
-
@haus-automatisierung
is klar -
@ben1983 sagte in [gelöst] Javascript-adapter ruft on ereignis doppelt auf:
@haus-automatisierung wann wird die Version im stable sein?
Heute freigegeben.
-
@haus-automatisierung Habe nun folgendes szenario:
shelly 2.5:
switch wird beschrieben mit true;
ich frage mit folgendem code ab, ob der shelly bestätigt hat:on({id:IdFrischwasserstellmotor, ack:true},(dp) =>{ if(dp.state.val) {...} }wird nun allerdings der wert des shelly.switch wieder auf true gesetzt,
so wird die Bestätigung (ACK) erst nach der in der Konfig eingestellten Abfragezeit gemeldet.
Ist das wirklich so gewollt?
Warum meldet der Adapter nicht direkt mit ack = true zurück?Ich hoffe es war verständlich.
-
@ben1983 sagte in [gelöst] Javascript-adapter ruft on ereignis doppelt auf:
Warum meldet der Adapter nicht direkt mit ack = true zurück?
Weil Du als Fehler gemeldet hattest, dass Du das nicht möchtest. Ich hatte alles sofort bestätigt, wenn ein Wert geändert wurde. Jetzt wird der erst bestätigt, wenn er vom Gerät kommt.
Ich würde Dich (erneut) bitten komplette Scripts zu zeigen und nicht nur Ausschnitte. Also inklusive Ansteuerung und was in
IdFrischwasserstellmotorsteht.Generell meldet der Shelly eigentlich die meisten Werte über MQTT und CoAP sofort zurück. Nur wenn etwas über HTTP abgerufen werden muss (weil es nicht per Event gesendet wird), kommt es zu der Verzögerung. Aber ohne Infos zu den genauen Datenpunkten usw. kann ich das natürlich nur raten.
Antwort: Es kommt drauf an.
-
@haus-automatisierung Nein, ichhatte gemeldet, dass er bei einmal wert setzen diesen 2 mal mit ack=true zurück gemeldet hat.
Er soll es ja direkt setzen, aber eben nur einmal mit true und nicht doppelt.Das ist das komplette script:
// Auswahl der Wasserquelle function chooseWaterSource() { // Check states for logging if((frischwasseranforderungZeit.val || frischwasseranforderungManuell.val || schwimmerschalter.val) && !_bypassZisterne.val) { SendTelegramMessage("Frischwasserbezug aktivieren",usernamePrivat); clearStateDelayed(IdZisternenstellmotor); clearStateDelayed(IdFrischwasserstellmotor); setState(IdZisternenstellmotor,true); } else { SendTelegramMessage("Zisternenwasserbezug aktivieren",usernamePrivat); clearStateDelayed(IdZisternenstellmotor) clearStateDelayed(IdFrischwasserstellmotor); setState(IdFrischwasserstellmotor,false); } } on({id:IdZisternenstellmotor, ack:true}, (dp) =>{ if(dp.state.val) { // Ausgabe des aktuellen States SendTelegramMessage("Zisternenstellmotor schließen",usernamePrivat); // Nachfolgend Frischwasserstellmotor öffnen setStateDelayed(IdFrischwasserstellmotor,true,verfahrzeitZisternenstellmotor.val * 1000); } else { SendTelegramMessage("Zisternenstellmotor öffnen",usernamePrivat); } }); on({id:IdFrischwasserstellmotor, ack:true},(dp) =>{ if(dp.state.val) { SendTelegramMessage("Frischwasserstellmotor öffnen",usernamePrivat); } else { // Ausgabe des aktuellen States SendTelegramMessage("Frischwasserstellmotor schließen",usernamePrivat); // Nachfolgend den Zisternenstellmotor öffnen setStateDelayed(IdZisternenstellmotor,false,verfahrzeitFrischwasserstellmotor.val * 1000); } });Zur erklärung, frischwasseranforderung Zeit wird durch eine shedule gesetzt.
Man kann es aber auch per Hand setzen. (FrischwasseranforderungManuell).
Wenn dies nun kurz hintereinander kommt, dann kommt es zu diesen überschneidungen.Ich habe nie behauptet, dass die Werte nicht direkt geschrieben werden sollen, das sollen sie ja sehrwohl,
aber sie wurden ja wie gesagt bei Änderung doppelt geschrieben und die beide male mit ack=true
Das war das eigentliche Problem.
So finde ich es aktuell etwas unvorteilhaft, da man die änderung erst nach x sekunden mit bekommt.Ps. In IdFrischwasserstellmotor / IdZisternenstellmotor stehen die Ids zu den shelly.switch states drin.
Also weißt Du was mein erstes Problem war?
EDIT:
Ich schalte quasi einen switch state eines shellys und reagiere dann darauf, wenn der shelly dies zurück meldet.
Wenn währenddessen etwas anderes angestoßend wird und dann der shelly erstverspätet zurückmeldet, dann gibt es durcheinander -
@haus-automatisierung sagte in [gelöst] Javascript-adapter ruft on ereignis doppelt auf:
@ben1983 sagte in Javascript-adapter ruft on ereignis doppelt auf:
Alle auf der neuesten Firmware: Datenpunkt "Relais0.switch" "Relais1".switch (Wobei ich andere jetzt nicht abfrage)
Fehler gefunden. Problem war, dass die Änderung einmal bestätigt wurde, wenn der Wert geändert wurde und zum Gerät geschickt wird. Zusätzlich antwortet das Gerät dann aber direkt über CoAP oder MQTT direkt mit dem neuen Wert. Was wiederum zu einem Event mit ack geführt hat.
Ist in 5.3.1 behoben.
Hier hast Du es doch selbst geschrieben.
Dachte Du bestätigst es jetzt wenigstens einmal.
Jetzt wird es ja gar nicht bestätigt, sondern nur nach der Zeit x... denke aber ein Schreiben des switchs (Ob Änderung oder nicht), sollte direkt bestätigt werden, aber eben nur einmalEdit: Bei Änderung von true auf false,oder false auf true, bestätigt der shelly adapter dies sofort.
Nur wenn der Wert schon passend steht, dann bestätigt er erst nach einer Zeit x.
Das wäre doch schön, wenn das auch direkt passieren würde. dann wäre es einheitlich -
@haus-automatisierung Habe es jetzt geändert und so gelöst.
Da wird halt nicht auf die Rückmeldung des shellys geachtet.// Auswahl der Wasserquelle function chooseWaterSource() { // Check states for logging if((frischwasseranforderungZeit.val || frischwasseranforderungManuell.val || schwimmerschalter.val) && !_bypassZisterne.val) { SendTelegramMessage("Frischwasserbezug aktivieren",usernamePrivat); clearStateDelayed(IdZisternenstellmotor); clearStateDelayed(IdFrischwasserstellmotor); // Zisternenstellmotor schließen setState(IdZisternenstellmotor,true); // Nachfolgend Frischwasserstellmotor öffnen setStateDelayed(IdFrischwasserstellmotor,true,verfahrzeitZisternenstellmotor.val * 1000); } else { SendTelegramMessage("Zisternenwasserbezug aktivieren",usernamePrivat); clearStateDelayed(IdZisternenstellmotor) clearStateDelayed(IdFrischwasserstellmotor); // Frischeasserstellmotor schließen setState(IdFrischwasserstellmotor,false); // Nachfolgend den Zisternenstellmotor öffnen setStateDelayed(IdZisternenstellmotor,false,verfahrzeitFrischwasserstellmotor.val * 1000); } } /***************************************************************************************** * ****************************************States***************************************** * ************************************************************************************ */ on(IdZisternenstellmotor, (dp) =>{ // Ausgabe des aktuellen States if(dp.state.val) { SendTelegramMessage("Zisternenstellmotor schließen",usernamePrivat); } else { SendTelegramMessage("Zisternenstellmotor öffnen",usernamePrivat); } }); on(IdFrischwasserstellmotor,(dp) =>{ // Ausgabe des aktuellen States if(dp.state.val) { SendTelegramMessage("Frischwasserstellmotor öffnen",usernamePrivat); } else { SendTelegramMessage("Frischwasserstellmotor schließen",usernamePrivat); } });Aber ich denke es sollte schon direkt zurück gemeldet werden mit ack= true, wenn der Wert geschrieben wird.
Nur dann eben nicht mehr durch CoAP oder MQTT ein zweites mal.Was denkst Du?
-
@ben1983 sagte in [gelöst] Javascript-adapter ruft on ereignis doppelt auf:
Aber ich denke es sollte schon direkt zurück gemeldet werden mit ack= true, wenn der Wert geschrieben wird.
Hängt natürlich von den Geräten ab, aber bei Z-Wave mach ich das z.B. nicht.
ack=truegibts da erst wenn vom Gerät die Änderung bestätigt wurde. Ansonsten kann man anhand des States ja gar nicht sehen ob es geklappt hat oder nicht. -
@alcalzone Ja klar, aber das gerät (shelly relais) meldet ja auch nicht 15s später erst zurück, dass es das relais angeschaltet hat. Es soll mir einfach zurückmelden, wenn es geklappt hat und nicht irgendwie zeitlich gesteuert.
Das ist aber aktuell nicht der fall. (Nur bei änderung des zustands. (ist dieser schon so aktiv, dann kommt die Meldung zeit gesteuert und ich denke das ist nicht ok, oder?
