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

Change to rtcm_msgs instead of mavros_msgs. #8

Merged
merged 1 commit into from Apr 4, 2023
Merged

Change to rtcm_msgs instead of mavros_msgs. #8

merged 1 commit into from Apr 4, 2023

Conversation

gsokoll
Copy link
Contributor

@gsokoll gsokoll commented Apr 4, 2023

This is in keeping with a similar PR for the LORD MicrosStrain NTRIP client where it is commented that
"rtcm_msgs is a smaller dependency with support for both ROS and ROS 2 now, and preferred by some downstream dependencies like ublox."

Format of the messages are structurally identical, with the only difference being the name of the payload ("data" in mavros vs "message" in rtcm_msgs)

This is in keeping with a similar PR for the LORD MicrosStrain NTRIP client.
LORD-MicroStrain/ntrip_client#34
where it is commented that
"rtcm_msgs is a smaller dependency with support for both ROS and ROS 2
now, and preferred by some downstream dependencies like ublox."

Format of the messages are strcturally identical, with the only difference
being the name of the payload ("data" in mavros vs "message" in rtcm_msgs)
@hortovanyi hortovanyi merged commit 61a6b8d into aussierobots:F9R-and-RTCM Apr 4, 2023
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.

None yet

2 participants