Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

GPS_STATUS: Fill in available messages for ROS1 legacy #1908

Merged
merged 2 commits into from
Oct 16, 2023

Conversation

SwiftGust
Copy link
Contributor

@SwiftGust SwiftGust commented Oct 12, 2023

Filled in available fields in GPS_RAW_INT & GPS2_RAW messages for ROS1

p.s. seems GPS2_RAW more complete than original GPS_RAW_INT

Filled in available fields in GPS_RAW_INT & GPS2_RAW messages

p.s. seems GPS2_RAW more complete than original GPS_RAW_INT
@SwiftGust SwiftGust changed the title GPS_STATUS Plugin: Fill in available messages for ROS1 legacy GPS_STATUS: Fill in available messages for ROS1 legacy Oct 12, 2023
ros_msg->h_acc = mav_msg.h_acc;
ros_msg->v_acc = mav_msg.v_acc;
ros_msg->vel_acc = mav_msg.vel_acc;
ros_msg->hdg_acc = mav_msg.hdg_acc
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lost ; here. See the CI build error.

@vooon vooon added this to the Version 1.18 milestone Oct 16, 2023
Copy link
Member

@vooon vooon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

@vooon vooon merged commit 7e7e2f6 into mavlink:master Oct 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants