ArduPilot / ArduPilot/MissionPlanner

LogBrowser: PIDS fields are all logged separately (because units are unknown?)

Open
#2,594 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
2.4k
Forks
2.9k
Avg merge
19h 16m
Merged PRs (30d)
4

Description

![image](https://user-images.githubusercontent.com/1498098/116193045-b1b89200-a769-11eb-84c0-4473f8ff6036.png)
Graphing the PIDS message is slightly difficult because each field appears with a difference scale.

The units for all fields are marked as unknown ([see code here](https://github.com/ArduPilot/ardupilot/blob/master/libraries/AP_Logger/LogStructure.h#L852)) so I think MP is assuming that all "unknowns" are different but I think we should assume they are all the same especially if they are all in the same message.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing how Mission Planner's LogBrowser assigns units and scales to PIDS fields, then compare that behavior with the linked AP_Logger/LogStructure.h definition around line 852. Confirm how fields from the same PIDS message are treated when their units are unknown. Done means the PIDS fields graph on a shared scale when appropriate, with existing log browsing behavior preserved.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
data-visualization
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.