Add PWM Dimmer multi-press and ledmask support #9584
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Restructure PWM Dimmer to support multi-press buttons.
Make PWM Dimmer function button hold time a constant .5 seconds allowing SetOption32 value to be used for determining hold time for non-PWM Dimmer functions.
Add support for ledmask command to specify which LEDs are used to indicate the brightness. LEDs cooresponding to unset bits in ledmask can be used for other purposes.
Add support for dimmer switches with no local load (no relay and no PWM - all buttons control remote devices).
Related issue (if applicable): fixes #
Checklist:
NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass