brutella / hkknx-public

hkknx is a HomeKit KNX bridge for KNX.
https://hochgatterer.me/hkknx
97 stars 6 forks source link

Bug Temperaturmesser #196

Closed chef76 closed 1 year ago

chef76 commented 1 year ago

Versucht man einen Temperaturmesser in eine Automation einzubinden erhält man nur abstruse Temperaturwerte: image

Macht man dasselbe mit einem Temperaturregler, passt es: image

Die Datentypen sind bei beiden definiert

Hier zum Vergleich ein EveWeather:

image

brutella commented 1 year ago

Tritt das Problem auch bei anderen HomeKit-Apps auf oder nur in der Apple Home-App?

chef76 commented 1 year ago

Auch bei anderen:

image

brutella commented 1 year ago

Ich vermute das Problem besteht, da für den Datentyp 9.001 in KNX ein Wertebereich von -273 °C ... 670 760 °C vorgegeben wird. Dieser Wertebereich wird jedoch in den HomeKit Apps nicht gut dargestellt.

Daher sollte der Wertebereich etwas kleiner gehalten werden. Welcher Bereich macht da Sinn? -50 ... 100 °C?

chef76 commented 1 year ago

Bist Du sicher, dass es daran liegt? Dann müsste das Problem bei Temperaturreglern doch auf auftreten oder hast Du da den Bereich begrenzt? Im Prinzip müssest Du meiner Meinung nach den Temperaturmesser genauso implementieren, wie es beim der Ist-Temperatur des Reglers gemacht ist.

Eine Beobachtung noch: Wenn ich die Automation mit demselben Wert in der Eve-App (der Screenshot oben ist aus der Koogeek-App)erstelle, dann werden die Temperaturen in der Eve-App korrekt angezeigt, z.B. 23 °. Öffnet man dann aber die erstellte Automation in Home, dann steht als Temperatur -4,5°

Ein sinnvoller Bereich wäre wohl -30° bis +50°

brutella commented 1 year ago

Bist Du sicher, dass es daran liegt?

Ja

Dann müsste das Problem bei Temperaturreglern doch auf auftreten oder hast Du da den Bereich begrenzt?

Bei Temperaturreglern wurden der Standardwertebereiche von HomeKit (0...100°C) nicht geändert. Bei Temperaturmessern schon.

Im Prinzip müssest Du meiner Meinung nach den Temperaturmesser genauso implementieren, wie es beim der Ist-Temperatur des Reglers gemacht ist.

Dort ist der Wertebereich 0...100°C. Das würde bei Temperaturmessern im Außenbereich nicht passen, da es Temperaturen unter 0°C auftreten können.

Eine Beobachtung noch: Wenn ich die Automation mit demselben Wert in der Eve-App (der Screenshot oben ist aus der Koogeek-App)erstelle, dann werden die Temperaturen in der Eve-App korrekt angezeigt, z.B. 23 °. Öffnet man dann aber die erstellte Automation in Home, dann steht als Temperatur -4,5°

Wie gesagt, der Temperaturwertebereich ist -273 ... 670 760 °C mit einer Auflösung von 0,01°C. Damit kommen halt die HomeKit Apps nicht klar.

Ein sinnvoller Bereich wäre wohl -30° bis +50°

Ist dieser Bereich nicht etwas zu klein? Es wird sicher vorkommen, dass Temperatursensoren in KNX mehr als 50°C messen.

chef76 commented 1 year ago

dann implementiere halt mal -50° bis +100°, dann sehen wir, obs funktioiert.

brutella commented 1 year ago

Ok, in 2.3.1-b2 kann man für die Ist-Temperatur -50...100°C oder -273 °C ... 670 760 °C auswählen. Für die Soll-Temperatur stehen 10...38°C, -50...100°C und -273 °C ... 670 760 °C zur Auswahl. Der Wertebereich 10...38°C ist in HomeKit Standard für die Soll-Temperatur.

chef76 commented 1 year ago

o.k, für die Temperaturmesser funktioniert es so, wenn ich -50 - 100 auswähle, allerdings kann ich in Homekit die Regler jetzt nicht mehr aufrufen. Wenn man einen Regler auswählt stürzt Home ab. Nur wenn man bei Ist-Temperatur = Soll-Temperatur = -50 -100 wählt, dann gehts wieder. Bei unterschiedlichen Datentypen krachts. Vielleicht kannst Du noch einbauen, dass der Datentyp beim Update automatisch gesetzt wird, denn sonst sehe ich das Forum schon wieder explodieren und ich hab auch keine Lust das jetzt bei allen meinen Reglern von Hand zu setzten. :-) Danke!

brutella commented 1 year ago

Nur wenn man bei Ist-Temperatur = Soll-Temperatur = -50 -100 wählt, dann gehts wieder

Hm, unter iOS 16 stürzt in der Home-App nichts ab.

Vielleicht kannst Du noch einbauen, dass der Datentyp beim Update automatisch gesetzt wird

Hab eine neuen Build von 2.3.1-b2 hochgeladen. Bei vorhandenen Geräten wird der Wertebereich jetzt nicht geändert. Das heißt bei Ist-Temperaturen ist der Wertebereich 0...100°C und bei Soll-Temperaturen 10...38°C. Also genau wie vorher, nur das die Ist-Temperaturen jetzt eine kleinerenn Wertebereich haben.

chef76 commented 1 year ago

Probier ich dann aus, aber das hatte ich eben auch von Hand eingestellt und dann hat's nicht funktioniert. 🤔

chef76 commented 1 year ago

Ok, die anderen Regler funktionieren wieder, aber die Anzeige in HKKNX passt nicht dazu:

image

Find ich mit dem Wissen, welches ich in dem Issue gesammelt habe etwas verwirrend, aber von mir aus ist es so o.k.

Danke!

brutella commented 1 year ago

Ok, die anderen Regler funktionieren wieder, aber die Anzeige in HKKNX passt nicht dazu:

Da hast du auch wieder Recht. Vielleicht sollte es gar keine unterschiedlichen Wertebereiche zum Auswählen geben. Ist-Temperatur ist immer von -50...100°C und Soll-Temperatur von 10...38°C. 🤔

chef76 commented 1 year ago

... für mich auch o.k. 👍

brutella commented 1 year ago

So, hab jetzt 2.3.1-b2 nochmal neu hochgeladen und die Wertebereiche sind jetzt so wie es im Release steht.