search

juergen-rocks / raumklima

Die berühmte Raumklimastation RS 500 von ELV Elektronik unter Linux mit einem Raspberry Pi auslesen.
MIT License
17 stars 12 forks source link

Develop; MQTT Feature #41

Closed MaStr closed 4 years ago

MaStr commented 4 years ago

Hallo, das ist der erste Versuch vom Pullrequest. Die für redis wurde zu einer generellen "Backend" ini geworden, da ja jetzt mehre möglich sind und über die Section in der Ini-Datei eine Abgrenzung möglich ist. Die "enabled" Funktion in der Ini hat noch keinen Effekt, ich möchte dies gerne dann in einer generellen "save2backend" Version verwenden.

Das Fehlerhandling im MQTT ist eher "Schönwetterflug". Und das Publish, welches asynchron läuft, sollte eigentlich auch erst nach allen publishes eingesammelt werden mit (wait_for_publish())-

Ich wollte aber schonmal den Stand als PR schicken und von dir Wünsche, Anregungen oder Änderungen einzusammeln.

edit: Ansonstn ist das jetzt erstmal wirklich nur ein "Senden" (Publish). Ergänzend wäre es sinnvoll dann Channel ohne Sensorwerte nicht zu bearbeiten und dann etwas mehr Logik reinzustecken um das dann auch im Topic abzubilden. Man könnte z.B. die gesendeten Nachrichten als "Retained" stehen lassen (und somit bekommt man immer den letzten Stand), mit der Ergänzung "Expiry" wird dann nach n Sekunden die Nachricht vom MQTT-Server verworfen.

Liebe Grüße Matthias

edelbluth commented 4 years ago

Ich werde heute Abend mit dem Review beginnen. Schonmal vielen Dank für die Arbeit!

edelbluth commented 4 years ago
edelbluth commented 4 years ago

Für Sourcecode-Formatierung und ein paar Kleinigkeiten habe ich Dir, @MaStr, einen Pull Request geschickt: https://github.com/MaStr/raumklima/pull/1

MaStr commented 4 years ago

Deine Vorschläge eingepflegt, angepasst und die README.md etwas erweitert. Ich hoffe, dass das so in deinem Sinn ist.

MaStr commented 4 years ago

Gerne!