CHAD-12259 zigbee motion: high battery usage #1142
Conversation
Test Results 55 files 352 suites 0s ⏱️ Results for commit 352d768. ♻️ This comment has been updated with latest results. |
|
zigbee-contact_coverage.xml
zigbee-motion-sensor_coverage.xml
Minimum allowed coverage is Generated by 🐒 cobertura-action against 352d768 |
|
I think I'll also be expanding this to Additionally, I think the monitored attribute health check might also still be a culprit even if the reporting configuration is updated. In that sense, the simple refresh from this PR should be an improvement but a rejoin will be required for a full fix, since I don't feel comfortable fully removing the monitoring of the IAS Zone attribute... |
greens
force-pushed
the
fixup/CHAD-12259
branch
from
4bec02a to
0a3216f
Compare
|
Channel deleted. |
| @@ -45,7 +45,6 @@ local zigbee_contact_driver_template = { | |||
| }, | |||
| sub_drivers = { | |||
| require("aqara"), | |||
| require("battery-overrides"), | |||
There was a problem hiding this comment.
Is this now released in 51?
There was a problem hiding this comment.
It wasn't for 51. It was just a sub-driver for one specific Visonic device. Behavior could have been captured by simply adding the model to drivers/SmartThings/zigbee-contact/src/contact-temperature-sensor/init.lua, so that's what I did.
Users reported higher battery consumption with edge drivers and a disparity was noted between DTH and driver cluster configurations. A forthcoming change in the lua libraries will change the defaults, but this change should significantly improve the situation for most users in the meantime with a simple refresh. Changes the IAS Zone reporting configuration to use the device defaults and changes temperature reporting to a max interval of 10 minutes with a 1 degree celsius reporting threshold. add change for zigbee contact as well
greens
force-pushed
the
fixup/CHAD-12259
branch
from
8c681a0 to
352d768
Compare
Users reported higher battery consumption with edge drivers and a disparity was noted between DTH and driver cluster configurations. A forthcoming change in the lua libraries will change the defaults, but this change should significantly improve the situation for most users in the meantime with a simple refresh. Changes the IAS Zone reporting configuration to use the device defaults and changes temperature reporting to a max interval of 10 minutes with a 1 degree celsius reporting threshold.