danielperna84 / hahomematic

Python 3 Interface for Home Assistant to interact with HomeMatic devices
MIT License
132 stars 21 forks source link

Invalid value 4.5 warning when turning off a thermostat #1787

Closed zapccu closed 1 week ago

zapccu commented 2 weeks ago

I agree to the following

The problem

When turning off a thermostat a value of 4.5 must be send by the integration to the CCU (30.5 is turning a thermostat permanentlly on). This results in the following log entry:

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht

Logger: hahomematic.platforms.generic.entity
Quelle: custom_components/homematicip_local/climate.py:405
Integration: Homematic(IP) Local (Dokumentation, Probleme)
Erstmals aufgetreten: 21. Oktober 2024 um 20:49:33 (1 Vorkommnisse)
Zuletzt protokolliert: 21. Oktober 2024 um 20:49:33

NUMBER failed: Invalid value: 4.5 (min: 5.0, max: 30.0, special:None)

Obviously the bounds checking of the integration has a problem. I guess this behaviour came in with the last update.

What version of HomematicIP (local) has the issue?

1.67.0

What was the last working version of HomematicIP (local)?

No response

What type of installation are you running?

Home Assistant OS

What type of installation are you running for your homematic backend?

CCU3

Which version of your homematic backend are you running?

3.77.7

What hardware are you running for your system?

Proxmox

Which config details do you use

Which interfaces do you use?

Diagnostics information (no logs here)

No response

Log file extract. Anything in the logs that might be useful for us? The log (Setting/System/Logs -> load full log) is the best source to support trouble shooting!

2024-10-21 20:49:33.497 WARNING (MainThread) [hahomematic.platforms.generic.entity] NUMBER failed: Invalid value: 4.5 (min: 5.0, max: 30.0, special:None)

Additional information

No response

SukramJ commented 2 weeks ago

No problem with my thermostats so i guess it's not related to the integration.

Which thermostat model? How do you turn off? What is the min temperature setting for the thermostat in CCU?

trilu2000 commented 2 weeks ago

I have exactly the same issue with a "Heizungsgruppe", as my radiator and wall mount thermostat is combined and visible in HA as "Heizungsgruppe". If i change the single devce, RT or WT all works as expected. The integration was working last winter, so i guess there was a change in the last time and only on "Heizungsgruppe" Please let me know if you need more information.

SukramJ commented 2 weeks ago

Andere Fragen hab ich nicht.

SukramJ commented 1 week ago

Wenn ich keine Antworten auf meine Fragen erhalte werde ich das Ticket schliessen.

trilu2000 commented 1 week ago

Bitte nicht schließen, ich hatte doch die Details geliefert. Zumindest aus meiner Sicht - WT und RD habe ich getestet, die gehen, aber Heizungsgruppen gehen nicht. Da tritt exakt der beschriebene Fehler auf.

SukramJ commented 1 week ago

Und wie Du an meiner Reaktion siehst sind für mich die Information nicht ausreichend / relevant! Was soll WT und RD sein?

Which thermostat model? How do you turn off? What is the min temperature setting for the thermostat in CCU?

trilu2000 commented 1 week ago

WT = Wallmount Thermostat, HM-TC-IT-WM-W-EU RT = Radiator Thermostat, HM-CC-RT-DN

Ausschalten einer Heizungsgruppe funktioniert mit keiner der beiden Möglichkeiten

Bei einem RT und WT funktioniert es ohne Probleme und ändert auch den Status der Heizungsgruppe.

In der CCU kann ich bei RT, WT und Heizungsgruppe 4.5 bis 30.5 Grad einstellen, bei 4.5 Grad ändert sich der Zustand auf Aus.

In Homeassistant kann ich bei RT, WT und Heizungsgruppe 5 bis 30.5 Grad einstellen. Wenn ich in Homeassistant bei RT und WT auf Aus drücke, dann schaltet das jeweilige Gerät aus und die Temperaturanzeige zeigt 4.5 Grad. Wenn ich in Homeassistant bei Heizungsgruppe auf Aus drücke, springt die Anzeige auf Heizbetrieb und
2024-10-24 13:46:15.571 WARNING (MainThread) [hahomematic.platforms.generic.entity] NUMBER failed: Invalid value: 4.5 (min: 5.0, max: 30.0, special:None) wird ins Log geschrieben.

SukramJ commented 1 week ago

Geht doch.

Bitte exportiere die Gerätedefinitionen der Heizgruppe mit dem Export-Service, und hänge bitte beide Dateien hier an.

(min: 5.0, max: 30.0, special:None)

Deutet darauf hin, das bei HM-Heizgruppen neuerdings 5.0 als minimaler Temperatur Wert steht.

trilu2000 commented 1 week ago

HM-CC-VG-1-device.json HM-CC-VG-1-paramset.json

SukramJ commented 1 week ago

Die Integration verwendet den erlaubten Wertebereich, den das Gerät liefert.

Der ist bei der Gruppe HM-CC-VG-1 5.0-30.0 und bei HM-TC-IT-WM-W-EU / HM-CC-RT-DN 4.5-30.0. Da die Homematic Definition 4.5 als Aus definiert funktioniert es bei der Gruppe nicht, da der Wertebereich nicht passt.

      "SET_TEMPERATURE": {
        "UNIT": "°C",
        "MIN": "5.0",
        "MAX": "30.0",
        "OPERATIONS": 7,
        "FLAGS": 1,
        "ID": "SET_TEMPERATURE",
        "TAB_ORDER": 4,
        "TYPE": "FLOAT",
        "DEFAULT": "20.0",
        "CONTROL": "HEATING_CONTROL.SETPOINT"
      },

Beim HM-TC-IT-WM-W-EU und beim HM-CC-RT-DN ist der Wertebereich 4,5-30.0


      "SET_TEMPERATURE": {
        "TYPE": "FLOAT",
        "OPERATIONS": 7,
        "FLAGS": 1,
        "DEFAULT": 20.0,
        "MAX": 30.5,
        "MIN": 4.5,
        "UNIT": "\u00b0C",
        "TAB_ORDER": 0,
        "CONTROL": "HEATING_CONTROL.SETPOINT"
      },
SukramJ commented 1 week ago

... und diese Validierung ist seid 11/2021 eingebaut.

SukramJ commented 1 week ago

Hat es mit der 1.66.0 noch funktioniert? Wenn möglich bitte mal testen.

trilu2000 commented 1 week ago

Eben getestet, Ja, funktioniert in der 1.66

SukramJ commented 1 week ago

@trilu2000 Vielen Dank. Wird in der nächsten Version gefixt sein. @zapccu Überleg dir das nächste mal, ob es Sinn macht ein Issue zu eröffnen, wenn Du keine Zeit/Lust hast mitzuarbeiten.

trilu2000 commented 1 week ago

Danke!

Baxxy13 commented 1 week ago

Eigentlich müsste eQ-3 die MIN/MAX - Definitionen für die HM-Heizgruppen an "den Standard" anpassen. Alle mir bekannten HM/HmIP - Geräte gehen von 4.5 - 30.5. Auch die HmIP-Heizgruppen. Nur die HM-Heizgruppen tanzen mit 5.0 - 30.0 aus der Reihe.

SukramJ commented 1 week ago

Eigentlich ... :wink:

zapccu commented 1 week ago

Sorry, dass ich nicht gleicht geantwortet habe. Auf der anderen Seite: dieser Bug in der Paramset-Description der CCU für Heizungsgruppen ist seit langem bekannt. Ich habe in meinem FHEM-Modul für die Homematic Integration eine entsprechende Korrektur irgendwann 2022 eingebaut.

Und einen Bug schließen, nur weil Infos fehlen, löst das Problem nicht.

P.S. das Problem mit der falschen Paramset Description gibt's auch bei anderen Geräten (z.B. bei einigen Rollladenaktoren mit Spezialwerten >1 für das LEVEL.

SukramJ commented 1 week ago

Und einen Bug schließen, nur weil Infos fehlen, löst das Problem nicht.

Ich schliesse Tickets wenn keine Reaktion kommt. Dann gehe ich davon aus, das kein Interesse mehr da ist.