Test Adapter Bayernluft 2.0.x
-
@boriswerner Hört sich gut an, und insbesondere, dass da die Zusammenarbeit mit dem Hersteller so gut läuft. Ich hatte mit Herr Akontz auch schon ein paarmal Kontakt per E-ail und Telefon, sehr kompetent und man merkt, er steht hinter seinen Produkten.
Schau mal bitte, bei Gelegenheit, ob der PR den ich gestern erstellt habe, so passt. War mei erstes Mal überhaupt. Damit kommen bei einigen Datenpunkten dann auch die Nachkommawerte mit rein.
Gruss, Jürgen
-
@wildbill Hi, sieht für mich soweit gut aus. Auf das community-Repo hab ich keinen Zugriff, das macht bisher ja mcm1957. Ich übernehme es aber auf jeden Fall in meine Überarbeitung für die ich dann wie beschrieben einen größeren PR stellen werde.
-
@Wildbill ich überlege gerade, ob es mehr Sinn ergibt den generellen Aufruf auf Dezimalpunkt umzustellen (siehe https://www.bayernluft.de/de/wlan32_changelist.html) :
WS32234901
Sowohl die Anzeige im Live-Schema, als auch die Export-Funktion für Smarthome-Systeme wurde auf vielfachen Wunsch um die Möglichkeit erweitert, als Dezimaltrennzeichen zwischen Punkt und Komma wählen zu können. Ohne weiteren Befehl werden die Werte weiterhin mit einem Komma ausgegeben. Möchte man als Trennzeichen einen Punkt, so hängt man den Befehl hinter den Aufruf.
Beispiel: http://192.168.178.190/index.html?decimal=point
oder
-
@boriswerner Das hatte ich sogar mal irgendwann gelesen und erfolgreich wieder verdrängt.
Ja, da parseFloat ja eh den Punkt als Dezimaltrenner erwartet, macht das vermutlich noch mehr Sinn. Dann kommt es gleich so, wie es soll und mein PR zum Umwandeln von Komma in Punkt wäre hinfällig. Wird wohl die bessere Lösung sein.
Gruss, Jürgen
-
@boriswerner Hab es bei mir lokal geändert und dann gleich einen PR erstellt und meinen alten geschlossen. Funktioniert einwandfrei.
-
@boriswerner sagte in Test Adapter Bayernluft 2.0.x:
so für neue Lüfter
Hallo
welche neue Lüfter?
hab ich da was verpasst? -
@negalein Wenn Du einen neuen Lüfter bei Bayernluft bestellst, dann wird da gleich die passende export.txt drauf sein. So habe ich es zumindest verstanden.
-
@wildbill said in Test Adapter Bayernluft 2.0.x:
@negalein Wenn Du einen neuen Lüfter bei Bayernluft bestellst, dann wird da gleich die passende export.txt drauf sein. So habe ich es zumindest verstanden.
So wird es sein. Für bestehende Geräte ändert sich da nicht viel (außer, dass es eine fertige Datei zum hochladen geben wird) Es gibt die Funktion direkt eine bestimmte export.txt anzusprechen. Die wird dann export_iobroker.txt heißen und der interne Aufruf an die API wird dann export=iobroker lauten.
Damit fällt für neue Käufer der Schritt weg, dass man erst im Dateibrowser jedes einzelnen Geräts die txt-Datei hochladen muss. Super Sache wie ich finde.
Für euch (bzw. alle, die bereits Lüfter haben) bedeutet das dann sobald die neue Version da ist, einmalig eine neue Datei auf jeden Lüfter hochladen. -
@wildbill sagte in Test Adapter Bayernluft 2.0.x:
neuen Lüfter
ahhhh, das ganze Gerät ist gemeint.
Ich hab an Ersatzlüfter (im Gerät) gedacht.
-
@mcm1957 Ich hab einen umfangreichen PR gestellt: https://github.com/iobroker-community-adapters/ioBroker.bayernluft/pull/83
Wenn du den mergen magst, kann der nochmal ausgiebig getestet werden (ich hab das schon ein paar Tage ohne Probleme im Einsatz), dann können wir auch mal das Latest angehen.Es gibt einen Breaking Change zu den bisherigen Versionen, es gibt nämlich eine komplett neue Datenpunkt-Struktur und es wird auch eine neue export-Datei auf den Geräten selbst benötigt. Für die Zukunft wird es dann aber wie schon beschrieben einfacher für neue Nutzer.
closed Redesign of the adapter (and fixed linting) #83
-
@boriswerner said in Test Adapter Bayernluft 2.0.x:
Hab dir schon im PR geantwortet. Hoff ich komm die Woche dazu. Da ja jedenfalls auch der Review fürs Repo offen ist kann man das auf einmal machen. Ev. sind ja diesbezüglich noch Änderungen notwendig.
Bitte häng mal im PR einen Objectdump (siehe https://github.com/ioBroker/ioBroker.repochecker/blob/master/OBJECTDUMP.md) an.
-
@boriswerner sagte in Test Adapter Bayernluft 2.0.x:
kann der nochmal ausgiebig getestet werden
ist dies in der 2.0.0-alpha.0 schon aufgenommen, oder ist noch was neues implementiert?gefunden
Breaking Change: All states from the 2.alpha versions or version 1 of the adapter were removed -
@negalein Es gibt eine komplett neue Datenpunktstruktur. An neuen Features ist "nur" die Korrektur der Dezimalstellen (siehe Kommentar von Wildbill) und das individuelle Setzen der einzelnen Lüfter bei ausgeschaltetem Gerät (ebenfalls von Wildbill eingebracht). Die Datenpunkte sind jetzt bzgl. der Rollen und Datentypen optimiert und insgesamt auch das Verhalten im Hintergrund wurde optimiert (check, ob ein Gerät überhaupt online ist, Fehlerbehandlung, etc.).
Also viel im Hintergrund aber auch ein bisschen neues -
@negalein
Nein@boriswerner hat einen größeren Umbau vorgenommen. Ich werde mir diesen im Laufe der Woche ansehen - wenn ich dazu komme. Derzeit ist diese Änderung noch nirgends in einem offiziellen Repo / im community Github enthalten. Da er die Struktur der States geändert hat wird was wohl eine 3.x.x werden müssen. Aber bevorich da eine Release einzeihe muss ich schaun was da wie geänder wird. Insbesondere warte ich auf den Dump der Objekte.
@Negalein
Da du wahrscheinlich noch die alte Version installiert hats, könntest du mir bitte auch einen Dump der jetzigen Struktur zum Vergleich zukommen lassen? Einfach hier oder an den PR anhängen. Anleitung siehe https://github.com/ioBroker/ioBroker.repochecker/blob/master/OBJECTDUMP.md.P.S:
Ich gehe davon aus dass die Lüfter LOKAL sind und daher in den Objekten keinerlei sicherheitsrelevante Daten drinnen sind. Aberbitte dies checken und diese ggf ausnullen/maskieren. -
@mcm1957 mein Dump ist hochgeladen. Die Geräte sind lokal, es sind MAC und die lokale IP enthalten, halte ich aber nicht für sicherheitsrelevant (oder?)
-
@boriswerner
Die IP sicher lokal (oder besitzt du ein öffentliche IPv4 Subnetz
). Die MAC würde ich auch nicht als kritisch sehen.
DANKE -
@mcm1957 sagte in Test Adapter Bayernluft 2.0.x:
bitte auch einen Dump der jetzigen Struktur zum Vergleich zukommen lassen?
-
Sorry fürchte wird die Woche nichts mehr mit Review. Hab mir Darminfekt eingefangen u lieg zZ flach
-
@mcm1957 Kurier Dich aus. Gute Besserung.
Gruss, Jürgen
-
Zuerst mal an dieser Stelle ein herzliches DANKE an @Boriswerner für die Anpassungen und Erweiterungen des Adapters.
Ich habe jetzt mal:
- die Änderungen von boriswerenr gemerged
- ein standardmäßiges Review für LATEST / BETA Repository durchgeführt (sie(he PR)
- diverse Anpassungen auf Grund des Reviews implementiert (siehe PR)
Das Resultat ist mal eine erste Testreleae 3.0.0-alpha.0.
Für den Test / Feedback habe ich mal einen 3.0.x Thread eröffnet:
https://forum.iobroker.net/topic/80006/test-adapter-bayernluft-3-0-x-alpha-testBitte dort weitermachen.
P.S: Die restlichen offenen Punkte werde ich hoffentlich zeitnahe erledigen.
