Skip to content

added support for SIM_STATE message#2073

Merged
vooon merged 4 commits intomavlink:ros2from
zekesarosi:ros2
Oct 14, 2025
Merged

added support for SIM_STATE message#2073
vooon merged 4 commits intomavlink:ros2from
zekesarosi:ros2

Conversation

@zekesarosi
Copy link
Copy Markdown
Contributor

We add support for the SIM_STATE message: https://mavlink.io/en/messages/common.html#SIM_STATE.
This message is helpful for developing autonomous flight modes and companion computer software.

Copy link
Copy Markdown
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, Tnaks!

But could you please add some more documentation into plugin brief and at each publisher creation?
Format is not firm yet, i just need something which i can reword later for #2068.

Comment thread mavros_extras/src/plugins/sim_state.cpp
Comment thread mavros_extras/src/plugins/sim_state.cpp
@zekesarosi zekesarosi requested a review from vooon October 13, 2025 18:28
Copy link
Copy Markdown
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 353741c into mavlink:ros2 Oct 14, 2025
3 of 4 checks passed
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