ArduPilot / WebTools

GNU General Public License v3.0
22 stars 17 forks source link

Improve webtools (mainly PID tool) documentation / useability #157

Open Janno117 opened 2 months ago

Janno117 commented 2 months ago

In the Filter review tool there's a description of what the tool does and what the tool needs in terms of logging data. If the data isn't present in the uploaded logs, the tool gives an error.

The PID-tool does neither have the text, nor the warning when there's not enough data in the logging. It just doesn't calculate the step-responses.

Proposed solution: Post a picture of the logging bitmasks with the minimum required logging options selected at the top of each tool. Alternatively post the parameter values of the needed logging parameter bitmasks with the minimum required options.

IamPete1 commented 1 month ago

I just have not had a chance to add that stuff yet. The step-response is a bit tricky, you can have plenty of data but still not get a result, the data also has to be good quality.

Log bitmask values would be nice, but the values you need a different for each vehicle type.