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

1.3 [Bug] New Routing with two units delivers message but sometimes the ACK returned is Max Retransmission Reached #1610

Closed
garthvh opened this issue Aug 8, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@garthvh
Copy link
Member

garthvh commented Aug 8, 2022

Seems to only happen when there are only two devices on the network.

In the image below all of the messages were received by the other device and shows on the device screen

IMG_2472

@garthvh garthvh added the bug Something isn't working label Aug 8, 2022
@GUVWAF
Copy link
Member

GUVWAF commented Aug 9, 2022

This might not be a bug, but something that may happen in an RF environment. If the implicit ACK somehow does not end up at the sender, it will do a retransmission. However, then the other node will not send an implicit ACK again, because it already did. I see in the source that only packets with want_ack will ACK a resent packet.
The reason why you only see this with two nodes in the network could be that then another node’s implicit ACK will end up at the sender.

@GUVWAF
Copy link
Member

GUVWAF commented Aug 11, 2022

feh123 reported this on Discord as well.
We could actually let nodes re-ACK a retransmission for broadcast messages as well, but we should be careful that we are not ACK-ing a rebroadcast again, as this would overload the channel. Right now, the receiving node will likely log “Ignoring incoming msg, because we've already seen it” when it receives the retransmission. This is good behavior for rebroadcasts, but maybe not for retransmissions. In order to re-ACK a retransmission, we could check whether the hop_limit is still at maximum, so we know it is the original sender that is retransmitting.
I would like to know whether this is the desired behavior, then I could try to implement and test this.

@garthvh
Copy link
Member Author

garthvh commented Aug 12, 2022

Seems especially pronounced in the same room while testing, I have not been able to replicate this very well with more distance between the nodes.

@garthvh
Copy link
Member Author

garthvh commented Aug 12, 2022

This sounds right to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants