SmartThingsEdgeDrivers icon indicating copy to clipboard operation
SmartThingsEdgeDrivers copied to clipboard

Matter Thermostat: Heat/cool setpoint and temp limits dynamic constraints

Open nickolas-deboom opened this issue 1 year ago • 4 comments

Subscribe to the temperature limit and heat/cool setpoint limit attributes and set capability constraints based on the limits.

nickolas-deboom avatar Feb 16 '24 18:02 nickolas-deboom

Channel deleted.

github-actions[bot] avatar Feb 16 '24 18:02 github-actions[bot]

Test Results

   57 files    366 suites   0s :stopwatch: 1 779 tests 1 779 :white_check_mark: 0 :zzz: 0 :x: 3 080 runs  3 080 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 03fe72b1.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Feb 16 '24 18:02 github-actions[bot]

File Coverage
All files 94% :white_check_mark:
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-thermostat/src/init.lua 94% :white_check_mark:

Minimum allowed coverage is 90%

Generated by :monkey: cobertura-action against 03fe72b195644b9bb70f517ac92ca39fcf3471e2

github-actions[bot] avatar Feb 16 '24 18:02 github-actions[bot]

The update made in cf3d46dee6736fb1545a771e32ab5b6760b304fc uses subscriptions for the temp range and setpoint range limits rather than reading them during init. This change was made to resolve an issue with InteractionQueueFull exceptions seen while onboarding a large number of devices at the same time.

nickolas-deboom avatar May 09 '24 19:05 nickolas-deboom

In the future, please squash commits.

greens avatar Jun 10 '24 19:06 greens