Skip to content

Comments

CHAD-17068: zigbee-humidity-sensor lazy load sub drivers#2622

Merged
aleclorimer merged 1 commit intomainfrom
zigbee-humidity-sensor-lazy-load-subdrivers
Jan 8, 2026
Merged

CHAD-17068: zigbee-humidity-sensor lazy load sub drivers#2622
aleclorimer merged 1 commit intomainfrom
zigbee-humidity-sensor-lazy-load-subdrivers

Conversation

@aleclorimer
Copy link
Contributor

Lazy loading of zigbee-humidity-sensorsub-drivers and copyright updates.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Channel deleted.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Test Results

   71 files    480 suites   0s ⏱️
2 483 tests 2 483 ✅ 0 💤 0 ❌
4 264 runs  4 264 ✅ 0 💤 0 ❌

Results for commit 2d69ed8.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

File Coverage
All files 95%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/aqara/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/init.lua 93%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/plant-link/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/frient-sensor/init.lua 93%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-humidity-sensor/src/frient-sensor/air-quality/init.lua 87%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 2d69ed8

Copy link
Contributor

@cjswedes cjswedes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The optimization to move the require "configurations" seems worthwhile on all these PRs because its often only used in device lifecycle handlers, and it contains a lot of constants, that can be gc'd when not in used.



local battery_defaults = require "st.zigbee.defaults.battery_defaults"
local configurationMap = require "configurations"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should move this intodevice_init to allow for gc.

@aleclorimer aleclorimer force-pushed the zigbee-humidity-sensor-lazy-load-subdrivers branch 2 times, most recently from 1fec820 to 4b2bbcc Compare January 8, 2026 20:22
@aleclorimer aleclorimer force-pushed the zigbee-humidity-sensor-lazy-load-subdrivers branch from 4b2bbcc to 2d69ed8 Compare January 8, 2026 20:45
@aleclorimer aleclorimer merged commit 9b50583 into main Jan 8, 2026
11 checks passed
@aleclorimer aleclorimer deleted the zigbee-humidity-sensor-lazy-load-subdrivers branch January 8, 2026 21:36
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.

2 participants