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

Get does not work with Saswell SEA801-Zigbee/SEA802-Zigbee #6796

Closed regissg closed 3 years ago

regissg commented 3 years ago

I have installed that device and I could set parameters like current_heating_setpoint or child_lock but I could not get any parameters.

For example zigbee2mqtt/TRV/get {"local_temperature": ""} gives error 2021-03-22 19:58:50: No converter available for 'local_temperature' ()

Here is an extract of the setup log data ..."interviewing":false,"model_id":"TS0601","network_address":24983,"power_source":"Battery","supported":true,"type":"EndDevice"},{"date_code":"","definition":{"description":"Thermostatic radiator valve","exposes":[{"access":1,"description":"Indicates if the battery of this device is almost empty","name":"battery_low","property":"battery_low","type":"binary","value_off":false,"value_on":true},{"features":[{"access":3,"description":"Enables/disables window detection on the device","name":"state","property":"window_detection","type":"binary","value_off":"OFF","value_on":"ON","value_toggle":"TOGGLE"}],"type":"switch"},{"features":[{"access":3,"description":"Enables/disables physical input on the device","name":"state","property":"child_lock","type":"binary","value_off":"UNLOCK","value_on":"LOCK"}],"type":"lock"},{"features":[{"access":3,"description":"Temperature setpoint","name":"current_heating_setpoint","property":"current_heating_setpoint","type":"numeric","unit":"Ā°C","value_max":30,"value_min":5,"value_step":0.5},{"access":1,"description":"Current temperature measured on the device","name":"local_temperature","property":"local_temperature","type":"numeric","unit":"Ā°C"},{"access":3,"description":"Mode of this device","name":"system_mode","property":"system_mode","type":"enum","values":["off","heat","auto"]},{"access":3,"description":"Offset to be used in the local_temperature","name":"local_temperature_calibration","property":"local_temperature_calibration","type":"numeric","unit":"Ā°C"},{"access":1,"description":"The current running state","name":"running_state","property":"running_state","type":"enum","values":["idle","heat"]},{"access":3,"description":"Away mode","name":"away_mode","property":"away_mode","type":"binary","value_off":"OFF","value_on":"ON"}],"type":"climate"},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"SEA801-Zigbee/SEA802-Zigbee","supports_ota":false,"vendor":"Saswell"},"endpoints":{"1":{"bindings":[{"cluster":"genBasic","target":{"endpoint":1,"ieee_address":"0x00124b000be89d05","type":"endpoint"}}],"clusters":{"input":["genBasic","genGroups","genScenes","manuSpecificTuya"],"output":["genOta","genTime"]},"configured_reportings":[]}},"friendly_name":"TRV","ieee_address":"0x847127fffe1b6875","interview_completed":true,"interviewing":false,"model_id":"TS0601","network_address":3682,"power_source":"Battery","supported":true,"type":"EndDevice"}]'

strandbein commented 3 years ago

First thank you very much for the great project. I have the same problem with SEA802-Zigbee. All /get commands end with:

ErrorĀ No converter available for 'get' 'window_detection' () ErrorĀ No converter available for 'get' 'child_lock' () ErrorĀ No converter available for 'get' 'current_heating_setpoint' () ErrorĀ No converter available for 'local_temperature' () ErrorĀ No converter available for 'get' 'system_mode' () ErrorĀ No converter available for 'running_state' () ErrorĀ No converter available for 'get' 'away_mode' ()

It is possible to send a redundant /set, then all values are also reported. But it would be more elegant with a functioning /get.

github-actions[bot] commented 3 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

strandbein commented 3 years ago

reopen, please

Koenkk commented 3 years ago

/get is not supported for TuYa based thermostats. (and it's unclear how to support it or if it is even possible since TuYa uses a custom protocol)

strandbein commented 3 years ago

ok, then the description should be adjusted here

Koenkk commented 3 years ago

Done, assuming this can be closed now.