Koenkk / zigbee2mqtt

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

Saswell SEA802-Zigbee local temperature calibration doesn't allow floats? #11261

Closed smalec closed 2 years ago

smalec commented 2 years ago

What happened?

It seems that it's not possible to set float value for local_temperature_calibration parameter. I'm not sure if this is a problem of software since (if I understand it correctly) this line indicates, that it should be possible with 0.1 step: https://github.com/Koenkk/zigbee-herdsman-converters/blob/master/devices/saswell.js#L49 I've tried to find device specification, but no luck so far :(

What did you expect to happen?

Publishing a message to zigbee2mqtt/FRIENDLY_NAME/set topic with payload {"local_temperature_calibration": 0.1} should set this parameter to 0.1 (or any information that device doesn't support float values).

How to reproduce it (minimal and precise)

  1. Publish a message to zigbee2mqtt/FRIENDLY_NAME/set topic with payload {"local_temperature_calibration": 0.1}:
    mosquitto_pub -t 'zigbee2mqtt/0x84fd27fffe335f8f/set' -m '{"local_temperature_calibration":0}'
  2. Verify somehow that state of device isn't correct (in web gui or in console):
    zigbee2mqtt      | Zigbee2MQTT:info  2022-02-06 16:05:56: MQTT publish: topic 'zigbee2mqtt/0x84fd27fffe335f8f', payload '{"anti_scaling":"ON","away_mode":"OFF","battery_low":false,"child_lock":"UNLOCK","current_heating_setpoint":22,"frost_detection":"ON","linkquality":255,"local_temperature":21.6,"local_temperature_calibration":-102,"preset_mode":"none","system_mode":"heat","window_detection":"OFF"}'

Zigbee2MQTT version

1.23.0

Adapter firmware version

0x26580700

Adapter

ConBee2/RaspBee2

Debug log

No response

brunoleon commented 2 years ago

Hello,

are you sure 0.1 is effectively supported ? I also have SEA-802 and the step is 1 not 0.1 See https://github.com/Koenkk/zigbee-herdsman-converters/pull/3657/files

smalec commented 2 years ago

No, I'm not sure. Probably you're right, and the step is 1. To avoid cnofusion I will create PR later.

gomble commented 2 years ago

I have the same Problem. Therefor my automatisation in Homeassitant to sync the local calibration with a thermometer does not work. How to solve this ?

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days