-
Notifications
You must be signed in to change notification settings - Fork 896
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
Introduce Link Statistics and Latency Measurement #477
Conversation
- Moved latency measurement from the radio driver to a new Latency class for better organization and readability. - Generalized the LinkStatistics class to allow for easy addition of other link statistics in the future.
If you want to play around with it, I suggest printing in the |
In a valiant attempt to appease the relentless gods of autopep8 and the all-seeing eyes of CI, I embarked on an epic journey to fix double-quoted strings and reorder imports.
connected to #469 |
- Modified start() and stop() methods for better thread management. - Implemented thread reinitialization on start after stop.
Anyway to attach a caller to this for this so that we can do a callback for each Crazyflie instance? Trying to find a good example to do that in a good way... the only one I could think of is the link_quality_callback but that's perhaps not the same level |
I would also like to discuss, before we merge, the ping interval, how many measurements we store, and if p95 is the best option |
Something like a |
Ah yes the console is a good option indeed. It's just that printing in a different thread gives issues sometimes, and I'd like to test it out with crazyswarm2 as well. |
Yeah I think adding a |
628bde6
to
4136a69
Compare
This PR introduces a framework for link statistics, enabling the tracking of various metrics, starting with latency measurement.
The implementation improves the lib's ability to monitor communication quality and lays the groundwork for further enhancements in link statistics.