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

Power reporting and decoupled mode for Aqara QBKG26LM #2734

Merged
merged 2 commits into from Jun 28, 2021

Conversation

ruifung
Copy link
Contributor

@ruifung ruifung commented Jun 28, 2021

Apparently the genAnalogInput cluster isn't present to be interviewed. Though it's sending attribute reports for it and attempting to GET the attribute just gives a no response error. Hence the ea.STATE and no tz.xiaomi_power here. (#1616)

Also the decoupled mode changes from #1986 (comment) , disableResponse due to observed behavior where it wont send a response, though it works, causing it to repeat configuration - and turning the switch off everytime z2m restarts.

This has been tested on my local installation and power reporting works, as does decoupled mode actions. Though it will always send actions even when not decoupled.

ruifung and others added 2 commits June 28, 2021 22:47
Apparently the genAnalogInput cluster isn't present to be interviewed. Though it's sending attribute reports for it and attempting to GET the attribute just gives a no response error. Hence the ea.STATE and no tz.xiaomi_power here.

Also the decoupled mode changes from Koenkk#1986 (comment)

This has been tested on my local installation and power reporting works, as does decoupled mode actions. Though it will always send actions even when not decoupled.
@Koenkk
Copy link
Owner

Koenkk commented Jun 28, 2021

Thanks!

@Koenkk Koenkk merged commit b412694 into Koenkk:master Jun 28, 2021
@ruifung ruifung deleted the patch-1 branch June 29, 2021 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants