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

[rtcm] RTK with UBlox8 modules #2034

Merged
merged 4 commits into from
Mar 18, 2017
Merged

[rtcm] RTK with UBlox8 modules #2034

merged 4 commits into from
Mar 18, 2017

Conversation

gautierhattenberger
Copy link
Member

This PR comes as a replacement on #1902
It is the same code with the NMEA mask fix and a fragmentation of packets to fit in RF modems if necessary.

@gautierhattenberger
Copy link
Member Author

@dewagter or any one from TUDelft, are you fine with these modifications ? (splitting the frames so it can work by default on XBee as well, configurable for bigger frames)

@flixr
Copy link
Member

flixr commented Mar 10, 2017

Basically LGTM, but the entries in .gitignore, .packages, control_panel_xml.example and default_rotorcraft.xml are missing compared to #1902

@gautierhattenberger
Copy link
Member Author

sorry I forgot them

@flixr flixr merged commit 3f4a05d into master Mar 18, 2017
@flixr flixr deleted the rtcm3 branch March 18, 2017 15:03
@flixr flixr mentioned this pull request Mar 18, 2017
1 task
podhrmic pushed a commit that referenced this pull request Mar 19, 2017
* [rtcm] RTK with UBlox8 modules

based on #1902, with fragmentation of packets to fit in RF modems if
necessary

* fix warnings

* [pprzlink] update pprzlink for RXMRTCM message

* [rtcm] add missing files
podhrmic pushed a commit that referenced this pull request Mar 19, 2017
* [rtcm] RTK with UBlox8 modules

based on #1902, with fragmentation of packets to fit in RF modems if
necessary

* fix warnings

* [pprzlink] update pprzlink for RXMRTCM message

* [rtcm] add missing files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants