Skip to content

Add MAV_SYS_STATUS_SENSOR_EXTENDED for remote leak sensor support#494

Merged
peterbarker merged 2 commits intoArduPilot:masterfrom
ES-Alexander:ap-add-leak-status
Mar 31, 2026
Merged

Add MAV_SYS_STATUS_SENSOR_EXTENDED for remote leak sensor support#494
peterbarker merged 2 commits intoArduPilot:masterfrom
ES-Alexander:ap-add-leak-status

Conversation

@ES-Alexander
Copy link
Copy Markdown

Port over mavlink#2427.

Includes the precursor PR to add the MAV_SYS_STATUS_SENSOR_EXTENDED bitmask more generally.

Comment thread message_definitions/v1.0/common.xml Outdated
ThomasDebrunner and others added 2 commits March 31, 2026 00:04
…tem (mavlink#1751)

Extend sys_status message with another triplet of bitfields to indicate sensor presence and health. Use last bit of existing fields to indicate presence of extended fields.
Copy link
Copy Markdown

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

... I'm guessing the commit structure isn't quite right now (referencing PRs where those PRs didn't actually make the changes involved).

I'm OK with that.

@peterbarker peterbarker merged commit 37d35a6 into ArduPilot:master Mar 31, 2026
12 checks passed
@peterbarker
Copy link
Copy Markdown

@ES-Alexander merged it!

I actually have patches here for to add the parachute stuff in....

@ES-Alexander ES-Alexander deleted the ap-add-leak-status branch March 31, 2026 04:20
@ES-Alexander
Copy link
Copy Markdown
Author

I actually have patches here for to add the parachute stuff in....

I hope I didn't inadvertently take an in-use value that wasn't officially claimed yet?
If so there's still time to change mine - my implementation PR hasn't been merged, so yours can still take priority.

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.

3 participants