Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated Moes TS0601 thermostat (_TZE200_b6wax7g0) #5207

Merged
merged 1 commit into from
Dec 27, 2022
Merged

Updated Moes TS0601 thermostat (_TZE200_b6wax7g0) #5207

merged 1 commit into from
Dec 27, 2022

Conversation

tefracky
Copy link
Contributor

The Moes TS0601 thermostat (_TZE200_b6wax7g0) has setpoints from 5 to 45 °C in 0.5 °C steps. Additionally, the systemMode can be 'off'.

@tefracky
Updated Moes TS0601 thermostat (_TZE200_b6wax7g0)
9b73c0a 
The Moes TS0601 thermostat (_TZE200_b6wax7g0) has setpoints from 5 to 45 °C in 0.5 °C steps. Additionally, the systemMode can be 'off'.
@Koenkk Koenkk merged commit 14583fd into Koenkk:master Dec 27, 2022
@Koenkk
Copy link
Owner

Koenkk commented Dec 27, 2022

Thanks!

@tefracky tefracky deleted the patch-2 branch December 29, 2022 15:59
@serlinGi
Copy link

serlinGi commented Jan 4, 2023 •
edited

hi,
another check on this update that doesn't fully match me.
changing the step to 0.5 is perfect (thx).
otherwise, the system_mode valve does't go to " off": once selected, it returns to heating status after a while.

Screenshot_2023-01-04-21-33-24-68_40deb401b9ffe8e1df2f1cc5ba480b12
Screenshot_2023-01-04-21-36-06-58_40deb401b9ffe8e1df2f1cc5ba480b12

@Heire
Copy link

Heire commented Jan 6, 2023

The off feature is available in the GUI but not working, it remains on heating:
Debug 2023-01-06 17:11:30Received MQTT message on 'zigbee2mqtt/Verwarming Keuken/set' with data '{"system_mode":"off"}'
Debug 2023-01-06 17:11:30Publishing 'set' 'system_mode' to 'Verwarming Keuken'
Info 2023-01-06 17:11:30MQTT publish: topic 'zigbee2mqtt/Verwarming Keuken', payload '{"battery":66,"boost_heating":"OFF","boost_heating_countdown":0,"boost_heating_countdown_time_set":300,"child_lock":"UNLOCK","current_heating_setpoint":20,"eco_mode":"OFF","eco_temperature":18,"linkquality":98,"local_temperature":21,"local_temperature_calibration":0,"max_temperature":25,"min_temperature":5,"position":25,"preset":"manual","programming_mode":"06:00/20°C 11:30/20°C 13:30/20°C 17:30/20°C 06:00/20°C 12:00/20°C 14:30/20°C 17:30/20°C 06:00/20°C 12:30/20°C 14:30/20°C 18:30/20°C","running_state":"heat","system_mode":"heat","update":{"installed_version":-1,"latest_version":-1,"state":"idle"},"update_available":null,"valve_state":"OPEN","window":"CLOSED","window_detection":"ON"}'

The heating_point doesn't accept the 0.5° and still rounds up to a round number:
Debug 2023-01-06 17:12:09Received MQTT message on 'zigbee2mqtt/Verwarming Keuken/set' with data '{"current_heating_setpoint":20.5}'
Debug 2023-01-06 17:12:09Publishing 'set' 'current_heating_setpoint' to 'Verwarming Keuken'
Debug 2023-01-06 17:12:10Received Zigbee message from 'Verwarming Keuken', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,21],"type":"Buffer"},"datatype":2,"dp":2}],"seq":1}' from endpoint 1 with groupID 0
Info 2023-01-06 17:12:10MQTT publish: topic 'zigbee2mqtt/Verwarming Keuken', payload '{"battery":66,"boost_heating":"OFF","boost_heating_countdown":0,"boost_heating_countdown_time_set":300,"child_lock":"UNLOCK","current_heating_setpoint":21,"eco_mode":"OFF","eco_temperature":18,"linkquality":98,"local_temperature":21,"local_temperature_calibration":0,"max_temperature":25,"min_temperature":5,"position":25,"preset":"manual","programming_mode":"06:00/20°C 11:30/20°C 13:30/20°C 17:30/20°C 06:00/20°C 12:00/20°C 14:30/20°C 17:30/20°C 06:00/20°C 12:30/20°C 14:30/20°C 18:30/20°C","running_state":"heat","system_mode":"heat","update":{"installed_version":-1,"latest_version":-1,"state":"idle"},"update_available":null,"valve_state":"OPEN","window":"CLOSED","window_detection":"ON"}'
Debug 2023-01-06 17:12:10Received Zigbee message from 'Verwarming Keuken', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,21],"type":"Buffer"},"datatype":2,"dp":2}],"seq":1}' from endpoint 1 with groupID 0
Info 2023-01-06 17:12:10MQTT publish: topic 'zigbee2mqtt/Verwarming Keuken', payload '{"battery":66,"boost_heating":"OFF","boost_heating_countdown":0,"boost_heating_countdown_time_set":300,"child_lock":"UNLOCK","current_heating_setpoint":21,"eco_mode":"OFF","eco_temperature":18,"linkquality":98,"local_temperature":21,"local_temperature_calibration":0,"max_temperature":25,"min_temperature":5,"position":25,"preset":"manual","programming_mode":"06:00/20°C 11:30/20°C 13:30/20°C 17:30/20°C 06:00/20°C 12:00/20°C 14:30/20°C 17:30/20°C 06:00/20°C 12:30/20°C 14:30/20°C 18:30/20°C","running_state":"heat","system_mode":"heat","update":{"installed_version":-1,"latest_version":-1,"state":"idle"},"update_available":null,"valve_state":"OPEN","window":"CLOSED","window_detection":"ON"}'

@tefracky
Copy link
Contributor Author

tefracky commented Jan 6, 2023

@Heire you are right, the 'off' state is not working, even if it should work in theory (determined by the Tuya) datapoints. Maybe this should be rolled back, I don't know, which is the best way to handle.

@serlinGi Thanks for the feedback! It seems, the 0.5 °C are not working for all decices. I can set the 0.5 °C at the device itself, but not over the online GUI. Unfortunately, I am new using ZigBee, so I also don't know, how to further test it...

@gitaen
Copy link
Contributor

gitaen commented Jan 10, 2023 •
edited

This TRV does not support 0.5 steps over zigbee

There is already an issue about Moes BHT-002-GCLZB thermostat and BRT-100-TRV not accepting 0.5 ºC steps via zigbee Koenkk/zigbee2mqtt#14965
It is also mentioned in #4832 (comment)

@gitaen
Copy link
Contributor

gitaen commented Jan 12, 2023

I think the TRV does not support {"system_mode":"off"}.
@tefracky did you find any way of setting it to OFF on the TRV itself?

@tefracky
Copy link
Contributor Author

I just pressed "-" and then the TRV goes to "OF"

20230112_123117

@gitaen
Copy link
Contributor

gitaen commented Jan 12, 2023

I just pressed "-" and then the TRV goes to "OF"

Thanks! I didn't know.

I set it off that way and looked at the received zigbee messages but it seems that state is not sent over zigbee.

debug 2023-01-12 14:28:27Received Zigbee message from 'Office TRV', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,5],"type":"Buffer"},"datatype":2,"dp":2}],"seq":7155}' from endpoint 1 with groupID 0 debug 2023-01-12 14:28:27Received Zigbee message from 'Office TRV', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,0],"type":"Buffer"},"datatype":2,"dp":5}],"seq":7411}' from endpoint 1 with groupID 0 debug 2023-01-12 14:28:27Received Zigbee message from 'Office TRV', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,88],"type":"Buffer"},"datatype":2,"dp":14}],"seq":7667}' from endpoint 1 with groupID 0

@tefracky
Copy link
Contributor Author

Yes, you are right. So maybe it would be better to remove the off state. Maybe someone else could comment on that and then I will decide.

@gitaen
Copy link
Contributor

gitaen commented Jan 18, 2023 •
edited

There is also a mention about this TRV not supporting off mode in this comment.
zigpy/zha-device-handlers#1123 (comment)

As this TRV supports neither 0.5 ºC steps over zigbee nor OFF mode, I think best way to go is to revert 14583fd.

@gitaen gitaen mentioned this pull request Jan 19, 2023
Closed
gitaen added a commit to gitaen/zigbee-herdsman-converters that referenced this pull request Jan 19, 2023
@gitaen
Revert "Add off system mode and improve current_heating_setpoint …
561d8b2 
…range for Moes BRT-100-TRV (Koenkk#5207)"

This reverts commit 14583fd.

Closes Koenkk#5370
@gitaen gitaen mentioned this pull request Jan 19, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tefracky @Koenkk @serlinGi @Heire @gitaen