search

KartoffelToby / better_thermostat

This custom component for Home Assistant will add crucial features to your climate-controlling TRV (Thermostatic Radiator Valves) to save you the work of creating automations to make it smart. It combines a room-temperature sensor, window/door sensors, weather forecasts, or an ambient temperature probe to decide when it should call for heat and automatically calibrate your TRVs to fix the imprecise measurements taken in the radiator's vicinity.
https://better-thermostat.org
GNU Affero General Public License v3.0
746 stars 123 forks source link

Saswell SEA801-Zigbee/SEA802-Zigbee overheats the room #135

Closed iksrd closed 2 years ago

iksrd commented 2 years ago

Installed the latest version. The temperature is set to 21.5. Saswell SEA801-Zigbee/SEA802-Zigbee overheat the room and turn off late. The room has time to heat up to 23.5. Is this normal?

platform: ai_thermostat name: kabinet_zapad_zapad thermostat: climate.0x804b50fffe52d26c temperature_sensor: sensor.a4c138c4c433_temperature valve_maintenance: false night_temp: 20 night_start: '22:00' night_end: '04:00' off_temperature: 21.5

platform: ai_thermostat name: kabinet_zapad_sever thermostat: climate.0xcc86ecfffe44c6ea temperature_sensor: sensor.a4c138c4c433_temperature valve_maintenance: false night_temp: 20 night_start: '22:00' night_end: '04:00' off_temperature: 21.5

platform: climate_group name: "Kabinet zapad" temperature_unit: C entities:

climate.kabinet_zapad_zapad climate.kabinet_zapad_sever

KartoffelToby commented 2 years ago

PLS checkout 0.9.1 i changed the rounding to round up insted of done.

The main problem is that the SEA801-Zigbee/SEA802 treated as a tuya device in Z2M and this uses a hex convertion for the values, sadly this current hex converter in Z2M isn't cabale of using float values, so only full int calibration is currently possible on this devices, also its limited to -6 - 6°C i think its a wrong limit because i have used higher values sucsessfull in test envs https://github.com/Koenkk/zigbee-herdsman-converters/pull/3527 But at this current state it means basicly that you have to deal with offsets to 0.9°C in the worst case.

KartoffelToby commented 2 years ago

There was also a main logic bug in the code, fixed in 0.9.2 its possible that the rounding is switched back to full round in the next versions.