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.

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

BRT-100-TRV null system_mode #14771

Closed
marcosimeone opened this issue Nov 2, 2022 · 6 comments · Fixed by Koenkk/zigbee-herdsman-converters#4903
Closed

BRT-100-TRV null system_mode #14771

marcosimeone opened this issue Nov 2, 2022 · 6 comments · Fixed by Koenkk/zigbee-herdsman-converters#4903
Labels
problem Something isn't working

Comments

@marcosimeone
Copy link

What happened?

Since last update (1.28.1) all BRT-100-TRV devices report null system_mode, making home assistant integration not properly functioning.

What did you expect to happen?

No response

How to reproduce it (minimal and precise)

No response

Zigbee2MQTT version

1.28.1 commit: 48117fd

Adapter firmware version

20210708

Adapter

zStack3x0

Debug log

log.txt

@marcosimeone marcosimeone added the problem Something isn't working label Nov 2, 2022
@callacomp
Copy link

the change came from this pull request. Koenkk/zigbee-herdsman-converters#4832

This has broken my TRVs also. They now say unknown. In the operation dropdown I can now only see heat (which is part of the update) but if I select it it changes back to unknown again, error in HA mqtt:
image
image
doesn't reflect in history or logbook
image

HA log
image

nothing in Z2m logs. altered heating, then temp, only logged temp change
Debug 2022-11-02 16:07:29Received MQTT message on 'zigbee2mqtt/Dining room TRV/set/current_heating_setpoint' with data '8.0' Debug 2022-11-02 16:07:29Publishing 'set' 'current_heating_setpoint' to 'Dining room TRV' Debug 2022-11-02 16:07:29Received Zigbee message from 'Dining room TRV', type 'commandDataResponse', cluster 'manuSpecificTuya', data '{"dpValues":[{"data":{"data":[0,0,0,8],"type":"Buffer"},"datatype":2,"dp":2}],"seq":14}' from endpoint 1 with groupID 0 Info 2022-11-02 16:07:29MQTT publish: topic 'zigbee2mqtt/Dining room TRV', payload '{"battery":88,"boost_heating":"OFF","boost_heating_countdown":0,"boost_heating_countdown_time_set":300,"child_lock":"LOCK","current_heating_setpoint":8,"eco_mode":"OFF","eco_temperature":17,"last_seen":"2022-11-02T16:07:29+00:00","linkquality":116,"local_temperature":17,"local_temperature_calibration":-1,"max_temperature":29,"min_temperature":6,"position":75,"preset":"manual","programming_mode":"11:06/18.5°C 14:32/21.5°C 11:30/22°C 17:30/23.5°C 07:02/22.5°C 12:01/23.5°C 12:31/22.5°C 16:31/21°C 04:01/20.5°C 17:28/20°C 13:30/21°C 18:30/20°C","running_state":null,"update":{"state":null},"update_available":null,"valve_state":"CLOSED","window":"CLOSED","window_detection":"OFF"}'

There's also no info in Z2M screen for new fields:
image

@leroyloren
Copy link

try to force a change on the valve, e.g. by switching programming/manual

@callacomp
Copy link

OK thanks @leroyloren , that works. I did it on two by the valve button and two from HA changing the mode, both updated the HVAC mode, and the running state seems more reliable now also, but this also needs a change of state to update, I think. It survived a few restarts also, might try a reboot later to see if it keeps this.

image
image

so I'll do them all now, but does this need a default value or something, or would it have updated if left?
image

@FredWst
Copy link

FredWst commented Nov 2, 2022

Hi,

Was in same case like u.
i made that : put in programming mode and back to manual, u'll get back to normal status. (manual & heat)

image

Fred

@xworld21
Copy link
Contributor

xworld21 commented Nov 2, 2022

I suspect this is working as intended... sorry about the confusion! As soon as preset changes, the valve will report the system_mode, and it should work smoothly from then onward. In hindsight, I could have added system_mode to the valve_state so that it updates more frequently, but after all this kind of glitch can only happen during an update – joining a new valve should return all properties right away.

Re the mqtt.climate error, I thought Home Assistant would work in optimistic mode about system_mode, i.e. if there is no set command, it thinks the change has gone ahead. But now I think I misunderstood the documentation. I'll look into this more carefully and see if there is a proper fix for this (either here on in Home Assistant).

@Koenkk
Copy link
Owner

Koenkk commented Nov 7, 2022

Fixed by Koenkk/zigbee-herdsman-converters#4903

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)

@Koenkk Koenkk closed this as completed Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
problem Something isn't working
Projects
None yet
6 participants