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

MAVLink Inspector: Add back Instance Awareness #12234

Open
hamishwillee opened this issue Dec 18, 2024 · 1 comment
Open

MAVLink Inspector: Add back Instance Awareness #12234

hamishwillee opened this issue Dec 18, 2024 · 1 comment

Comments

@hamishwillee
Copy link
Contributor

MAVLink messages that have an "instance field" should support separate plotting and reporting for each value of the instance field.

As an example consider the BATTERY_STATUS message. This has the id field with the instance="true" attribute as shown below:

<field type="uint8_t" name="id" instance="true">Battery ID</field>

If a vehicle has multiple batteries then each of these should have a different value for id and should be plotted separately. As it is now a plot of BATTERY_STATUS is useless for multi-battery systems, because the message plot shows traces that switch between different sources.

Currently this affects batteries, ESC, IMUs, but there are others, and more to come.

Note that this was previously supported, but was lost when transitioning to QML controls.


Tell us a bit about the feature:

  • What problem does it solve?

    Allows instance data to be plotted and inspected.

  • What does it do?

    Allows data to be presented usefully

  • What flight stacks must it work with? (All, PX4, ArduPilot)

    All

  • Relevant vehicle types? (All, multirotor, fixed-wing, VTOL, submarine, etc.)

    All

  • Are there other systems that have this feature?

    Mission Planner.

  • What communication/integration standards does the feature rely on (e.g. MAVLink commands etc.)

    MAVLink

@DronecodeBot
Copy link

This issue has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:

https://discuss.px4.io/t/custom-mavlink-messages-for-multiple-instances/42597/21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants