Some widgets cant display ESC telemetry
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.
The fields are selectable correctly in the plotter widget.
Steps to reproduce
- Connect a droneCAN or similar ESC that supports telemetry
- Add a very generic indicator widget
- 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.
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!
Sounds great! Thanks for the heads up
@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.
#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.