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

Improve the Crazyflie MAVLink tunnel to increase efficiency #9245

Merged
merged 2 commits into from
Apr 5, 2018

Conversation

LorenzMeier
Copy link
Member

@LorenzMeier LorenzMeier commented Apr 5, 2018

This change fragments MAVLink packets more efficiently and therefore increases the net throughput. This in turn makes the connection significantly more stable and the Crazyflie experience overall more usable.

Replaces #9232

This change fragments MAVLink packets more efficiently and therefore increases the net throughput. This in turn makes the connection significantly more stable and the Crazyflie experience overall more usable.
@LorenzMeier
Copy link
Member Author

@barzanisar I cleaned the commits and I reverted the unintended submodule changes in #9232. Compare the diff of the two PRs to observe what I removed. Make sure to commit changes per files and never use things like commit -a or alike, as that includes stale submodules or other files.

@LorenzMeier
Copy link
Member Author

@barzanisar Do you have an issue to track the remaining Crazyflie changes? Where are the QGC and other changes?

@barzanisar
Copy link

barzanisar commented Apr 5, 2018

@LorenzMeier
Issue for adding a parameter to change Joystick message frequency from QGC is discussed in mavlink/qgroundcontrol#6297

Issue for changes to cfbridge are discussed in
dennisss/cfbridge#7

Issue for changes to crazyflie-lib-python are discussed in bitcraze/crazyflie-lib-python#75

The changes that solve the issues dennisss/cfbridge#7 and bitcraze/crazyflie-lib-python#75 are combined in the crazyflie-lib-python fork .

The instructions for using the whole crazyflie setup including the cfbridge are documented in the PX4 user guide.

@LorenzMeier LorenzMeier merged commit d025eb0 into master Apr 5, 2018
@LorenzMeier LorenzMeier deleted the barzanisar-pr-crazyflie branch April 5, 2018 16:11
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