search

Koenkk / zigbee2mqtt

Zigbee 🐝 to MQTT bridge 🌉, get rid of your proprietary Zigbee bridges 🔨
https://www.zigbee2mqtt.io
GNU General Public License v3.0
11.72k stars 1.64k forks source link

SEA802-Zigbee Local Temperatur calibration error #11777

Closed gomble closed 2 years ago

gomble commented 2 years ago

What happened?

I am getting following error:

Logger: homeassistant.components.mqtt.number Source: components/mqtt/number.py:191 Integration: MQTT (documentation, issues) First occurred: 23:20:05 (16 occurrences) Last logged: 23:45:21

Invalid value for number.thermostat_esszimmer_local_temperature_calibration: -32.0 (range -30.0 - 30.0) Invalid value for number.thermostat_esszimmer_local_temperature_calibration: -33.0 (range -30.0 - 30.0)

What did you expect to happen?

No response

How to reproduce it (minimal and precise)

No response

Zigbee2MQTT version

1.24.0

Adapter firmware version

0x26720700

Adapter

ConBee2/RaspBee2

Debug log

No response

Koenkk commented 2 years ago

Try setting the local_temperature_calibration once via the frontend (so it comes in the -30 till 30 range)

gomble commented 2 years ago

Try setting the local_temperature_calibration once via the frontend (so it comes in the -30 till 30 range)

the problem is that i can only go -6 to 6 celsius if i go more then -6 or 6 then the temperature does not change. But the slider on the frontend goes from -30 to 30 Also some automatisations in homeassistant do not work to sync the trv with temperature sensor

Koenkk commented 2 years ago

Changed the limits to -6 - 6 now.

Changes will be available in the dev branch in a few hours from now. (https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html)

gomble commented 2 years ago

Okay one more info the calibration does not work for numbers with decimal This does not change the temperature: image But this is working: image

Possible numbers are: -6,-5,-4,-3,-2,-1,0,1,2,3,4,5,6

I will test it in the dev branch

Koenkk commented 2 years ago

@gomble also updated the step size.

gomble commented 2 years ago

Hey that worked image

gomble commented 2 years ago

when this will be merged to a release ?

Koenkk commented 2 years ago

Next release (on 1 April)

ItsMisterP commented 1 year ago

Hey @gomble do you investigate, why this aint work with decimals? Regards Philip

gomble commented 1 year ago

Hey @gomble do you investigate, why this aint work with decimals?

Regards

Philip

Yes it was fixed in the next release But you can only calibrate from -10 to 10

ewolf29 commented 1 year ago

Are we certain that decimal step sizes are not supported by the device? Decimal number calibration does actually not make any sense in this case.

I would volunteer to test it if it gets changed on the dev branch.

Or does 54d8241f491c5ed3e28eb36b91803b71005ebbfb simply change to allowed stepsize in the GUI while I still can set it to decimal values using MQTT messages?