ahoy icon indicating copy to clipboard operation
ahoy copied to clipboard

add displ_motion feature in API

Open DanielR92 opened this issue 1 year ago • 1 comments

Feature request and added now in API. Please check if is possible to add in mqtt...

Feature requested here: https://github.com/lumapu/ahoy/issues/1633

DanielR92 avatar Aug 30 '24 07:08 DanielR92

Hier einmal ein Alternativvorschlag:

Edit 3.9.2024: https://github.com/You69Man/ahoy/commits/feature/PIR_to_MQTT/ (wurde gelöscht!) Neu: https://github.com/You69Man/ahoy/commits/feature/PIR_to_REST_and_MQTT/ Den commit [remove getGeneric from getSetup (getSetup nests getSysInfo, which already nests getGeneric on its part)] kann man gerne auch skippen, der hat eigentlich mit der Sache nichts direkt zu tun, sondern könnte allenfalls eine kleine Optimierung der RestAPI sein, die ohnehin etwas überanstrengt ist... Vielleicht hat sie aber auch unerwünschte Nebenwirkungen

Es werden die Werte "display_active" sowie "motion_sensor_active" über die Seite /api/generic ausgegeben. Die Dauer von "motion_sensor_active" hängt von der Verzögerungsdauer des PIR ab. Bei manchen PIR Modulen ist diese über ein Poti einstellbar, bei anderen nicht. Bei letzteren kann es sein, dass die Zeit sehr kurz ist (nur wenige Sekunden), dann würde man sie nicht sehen, wenn man die RestAPI z.B. nur alle 10 Sek. abfrägt. "display_active" zeigt an, ob das Display eingeschaltet ist. Wenn allerdings die Display Konfiguration "Schalte das Display aus, während die Wechselrichter aus sind", ist der Wert natürlich auch aus, und der Einbrecher kann ungestört durchs Haus gehen ;) Es gibt nun im App-Interface Zugriffsfunktionen auf die zugehörigen Variablen, damit sollte es grundsätzlich auch kein Problem sein, diese auch über MQTT zur Verfügung zu stellen, aber diese Einbindung lassen wir besser @lumapu, sonst gibt's wieder irgendwelche Probleme mit MQTT :)

Edit 4.9.2024: Habe mich nun doch selbst an den MQTT Publish gewagt. Bei mir funktionierts tadellos, ist aber mein erster Gehversuch mit MQTT, daher bitte reviewen, ob das so passt!

You69Man avatar Sep 02 '24 19:09 You69Man