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

Add Synchronize Capability #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Synchronize Capability #1

wants to merge 1 commit into from

Conversation

btli
Copy link

@btli btli commented Nov 28, 2021

Set a time interval to retransmit open/close command to synchronize shade status. Useful for poor RF conditions since there's no 2-way communications for this protocol.

Set a time interval to retransmit open/close command to synchronize shade status. Useful for poor RF conditions since there's no 2-way communications for this protocol.
@Gassgs
Copy link
Owner

Gassgs commented Nov 29, 2021

This doesn't account for the position correct? I think if you want to retransmit the command it should be the position not only either 100% open or 0% closed?? IDK, Is this change working well for you? I've never had an issue with commands getting missed.

The bridges are so cheap it would probably be better to get another one so they are close to the devices you are trying to control...

@btli
Copy link
Author

btli commented Nov 29, 2021

Maybe my use case is a bit different. Multiple shades are controlled with a single command. Some shades are outdoor too (the RF bridge won't survive long out there). On occasion a shade will miss the signal.

Not entirely sure it's possible to synchronize position either since everything is based on timing. Without a confirmation from the receiving device, sending a set position would just cause the shade to run further in one direction or the other -- even if it received the original command.

Works for me, but it does create excess events.

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