search

netz39 / rollladensteuerung

Schaltpläne, Layouts und Firmware für Schließanlage und Rollladensteuerung
http://www.netz39.de/wiki/projects:2014:gatekeeper
0 stars 6 forks source link

Python-Script für den Tür-Watchdog #14

Closed penguineer closed 5 years ago

penguineer commented 5 years ago

Löst das alte Shell-Script ab und ist damit auch kompatibel zum Script von @maz3max .

Das Verhalten ist im Prinzip wie beim alten Script, allerdings wird die State Machine nun vollständig und sinnvoll abgelaufen.

Da eine passende Anbindung fehlt, kann dieses Script derzeit nicht piepsen.

penguineer commented 5 years ago

(Update: Rebase auf aktuellen Master)

24367dfa commented 5 years ago

Das sieht so weit für mich verständlich und nachvollziehbar aus. Ich muss mir die Statemachine nachher nochmal aufmalen, aber davon ab, passt das so denke ich.

Abgesehen davon wäre es vielleicht gut, ein bisschen was von der Dokumentation (Statemachine/Messages etc) von unserem Wiki mit hierher umzuziehen, damit die Dinge ein weniger besser verknüpft sind (und bleiben).

penguineer commented 5 years ago

Danke für das Review und das positive Feedback! :) Ich merge den PR auf der Basis.

penguineer commented 5 years ago

Über die Dokumentation habe ich auch schon nachgedacht. Allerdings ist das projektübergreifend. Man könnte in der Rollladensteuerung die Dinge dokumentieren, die für die Rollladensteuerung relevant sind. Dann gibt es aber auch noch die Space Notification (und beide greifen aufeinander zu).

Eigentlich bräuchte man ein System, das sich die Fragmente aus den jeweiligen Projekten zuammensuchen kann, damit es irgendwo auch eine Übersicht gibt. Auf jeden Fall etwas für ein eigenes Ticket oder ein eigenes Projekt.

Grundsätzlich stimme ich aber zu, dass zentraler und gleichzeitig näher am Code dokumentiert werden sollte.