cockpit icon indicating copy to clipboard operation
cockpit copied to clipboard

Some widgets cant display ESC telemetry

Open mc-nugget opened this issue 9 months ago • 3 comments

Bug description

The messages ESC_TELEMETRY_1_TO_4 and ESC_TELEMETRY_5_TO_8 display as NaN when used in the very generic indicator widget.

Image

The fields are selectable correctly in the plotter widget.

Image

Steps to reproduce

  1. Connect a droneCAN or similar ESC that supports telemetry
  2. Add a very generic indicator widget
  3. Try to assign a field from the ESC telemetry mavlink messages

Primary pain point(s)

No response

Additional context

No response

Prerequisites

  • [x] I have checked to make sure that a similar request has not already been filed or fixed.

mc-nugget avatar Mar 26 '25 12:03 mc-nugget

Hey @mc-nugget, thanks for the report!

It looks like this is happening because the Plotter widget is using the data-lake system, which is correctly parsing the /rpm value as an array. You can see that by lokking at the Plotter options, where you can select /rpm/1, while in the VeryGenericIndicator you could only select /rpm, which is an array and thus not a number.

We will be moving the VeryGenericIndicator widget to the data-lake system in the next weeks as well, and with that I expect this bug to be fixed!

rafaellehmkuhl avatar Mar 26 '25 13:03 rafaellehmkuhl

Sounds great! Thanks for the heads up

mc-nugget avatar Mar 26 '25 16:03 mc-nugget

@mc-nugget I remembered I was already working at this, so I rebased the branch I had and put a little more effort on making it work. Watch #1778 to be updated when this gets merged.

rafaellehmkuhl avatar Mar 26 '25 17:03 rafaellehmkuhl

#1778 was merged a while ago, so closing.

Please comment if the issue is not actually fixed, and we can re-open and try to investigate as relevant.

ES-Alexander avatar Jul 16 '25 11:07 ES-Alexander