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 disconnect timeout counter for the radio link #428

Closed
ataffanel opened this issue May 3, 2019 · 2 comments

Comments

Projects
None yet
2 participants
@ataffanel
Copy link
Member

commented May 3, 2019

In order to allow #393 to be closed and also to fix connections problems where the Crazyflie already has a brunch of log block running and the lib is flooded with the log blocks when connecting it would be good to have a timeout on radio packets that indicate that the radio link is disconnected. When the link is disconnected, the log subsystem will reset itself which solves both USB problems and radio connection problems.

This radio timeout can be set to 1 seconds.

@whoenig this means that if the Crazyflie does not receive any packet (including Null packets and broadcasts) for more than 1 seconds, the log subsystem will reset. Is that going to be a problem for ROS/Crazyswarm usecase?

@whoenig

This comment has been minimized.

Copy link
Contributor

commented May 3, 2019

This shouldn't cause any issues. If logging is enabled, we frequently send empty ping messages (if no other messages are sent). If logging is disabled (like in the Crazyswarm by default), it shouldn't matter if the log system resets itself.

@ataffanel

This comment has been minimized.

Copy link
Member Author

commented Jun 5, 2019

OK, current implementation seems to work fine so lets close this issue.

@ataffanel ataffanel closed this Jun 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.