-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
1.16.2 has broken by Tuya dimmer gq8b1uv compared to 1.16.1 #6366
Comments
|
|
Could you provide the herdsman debug log when sending this command with both 1.16.1 and 1.17.1? To enable herdsman debug logging, see https://www.zigbee2mqtt.io/information/debug.html#zigbee-herdsman-debug-logging |
OK, this was real pain to collect under Windows :) I managed to set the environment variable and run z2m by entering Anyway, non working 1.17.1
And working 1.16.1
|
Thanks, I've investigated but I don't see the problem yet. The produced bytes are exactly the same. Would you be also able to sniff the traffic when sending this command? When doing so make sure to set exactly the same brightness in each sniff and do it in the following sequence (to make sure variables like transaction id which is incremented by one with every command is the same).
https://www.zigbee2mqtt.io/how_tos/how_to_sniff_zigbee_traffic.html |
OK, will do, but I will be away for a few days. I should have the capture ready by next weekend. Thanks for your effort so far. |
While preparing for capture, I was manually sending the MQTT command to topic However, publishing payload Can you make something out of this or would you still require packet captures?
|
OK, please find attached two packet captures using 1.16.1 and 1.17.1. These are results of publishing
|
I did not look at the sniff yet. But given it work when setting only the Above https://github.com/Koenkk/zigbee-herdsman-converters/blob/2395e4445a9beb605a823e4767fec6710d5658bd/lib/tuya.js#L396 can you try adding: |
It works fine with the above modification! Both using slider and publishing
|
Does it only work with |
It also works fine with setting that to 1, 0 or seemingly any other value. |
Integrated the fix now, please let me know if it works. Changes will be available in the latest dev branch tomorrow (https://www.zigbee2mqtt.io/how_tos/how-to-switch-to-dev-branch.html) |
It works now, thanks!! 👍 |
What happened
Just noticed (using latest 1.17.1) that brightness controll on my Zigbee smart dimmer (gq8b1uv) does not work. Moving the slider in Home Assistant does not change brightness. In z2m logs, after every slider move, it just sends a message with brightness 254. On-off controll works fine.
One more thing to note is that "Effect" propery is discovered through Home Assistant in the affecting z2m versions. This device does not support effects.
Comparing the versions, it seems that a large change was introduced to "tuya_dimmer_level" function within toZigbee.js file. I don't know if that is related.
Note that reporting works fine - rotating knob on the dimmer manually also moves the brightness slider in HA, but not vice versa. See logs.
What did you expect to happen
Brightness to work properly as it did on 1.16.1 (last version where it worked)
How to reproduce it (minimal and precise)
Debug info
Zigbee2MQTT version: 1.16.2 through 1.17.1
Adapter hardware: CC2531
The text was updated successfully, but these errors were encountered: