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

Support for dimmable lights & improved shutter control #6

Merged
merged 8 commits into from
Sep 21, 2022

Conversation

optiluca
Copy link
Contributor

I've made a few modifications to support:

  • Dimmable lights
  • Shutters. These were already nominally supported, but in my setup they were a bit unreliable. The "stop" button didn't work, (as the STATE_OPENING, STATE_CLOSING states were not handled), and if a shutter was left in an intermediate state it could think it was already closed. This new implementation errs on the side of caution, always ensuring that all shutters are closed when a close command is sent.

I have some doubts about any breakages I may have introduced in features I don't use, but figured I'd share my changes with you in case you were interested in any of it.

Grazie!

Luca

…htness state. Also assumes that all bulbs support brightness...
…mable and not, and found the right permutation of states to ensure that the brightness state is correctly handled.
… become clear that the implementation is intrinsically poor. We should be 1) fixing the position sensing upstream and 2) monitoring that.
…to just be position, but don't trust it yet...
@gicamm gicamm merged commit af5d581 into gicamm:master Sep 21, 2022
@gicamm
Copy link
Owner

gicamm commented Sep 21, 2022

Hi Luca,
I'm sorry for the considerable delay.
Something wrong happened with notifications.
Please feel free to do any kind of improvements.

Grazie a te!
Giovanni

@gicamm
Copy link
Owner

gicamm commented Sep 21, 2022

@optiluca

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