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

More intelligent send retry management with ACK #495

Open
asoltesz opened this issue Jan 2, 2020 · 1 comment
Open

More intelligent send retry management with ACK #495

asoltesz opened this issue Jan 2, 2020 · 1 comment

Comments

@asoltesz
Copy link

asoltesz commented Jan 2, 2020

Currently, even if ACK is set, the code will not check if the sending was successful and resends the message as many times as specified with setRetries().

A better and more logical behavior would be if ACK and retries worked together and sending retries would only happen if no ACK was received from the gateway for the message. That would also decrease power consumption in comparison with unconditional retries.

@user2684
Copy link
Contributor

user2684 commented Jan 5, 2020

Hi, interesting suggestion. But I wondering if feasible outside the MySensors library (e.g. in NodeManager) or if it is something supposed to be part of the core code. Do you have by chance a sample code to share just to understand better? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants