Tools: Extract parameter default values from an ArduPilot .bin log file

[amilcarlucas]

Contains unittests with 96% coverage

@peterbarker how should I integrate the unittests with the autotest suite?

[amilcarlucas]

To run coverage do:

cd Tools/autotest/param_metadata
python3 -m coverage run -m unittest extract_param_defaults_unittest.py
python3 -m coverage html
firefox htmlcov/extract_param_defaults_py.html

[khancyr]

we don't gather coverage for python code for now. Adding it should quite simple, generate the coverage report and put it in the upload directory for coveralls. Then it should appears online.

[amilcarlucas]

Thanks Pierre,

My question is how to integrate the unit tests so that they run every time the extract_param_defaults.py or extract_param_defaults_unittest.py file changes?

Coverage information is just a cherry on top. Nice but not important now.

[IamPete1]

I don't think its a good idea to provide defaults files, this makes it harder for us to change the defaults going forward as users will now have out of date files they will be comparing to. The defaults are also only valid in the particular configuration that the parameters are in. So the defaults are only correct with the context of the set parameter values.

What problem is this trying to solve? Mission planner now shows the defaults for all parameters in its full list.

[amilcarlucas]

I am trying to solve the problem of managing (without MP) multiple sets of parameters for multiple vehicles.
I want to store only the parameters that change relative to the defaults of a particular vehicle in a particular configuration in a particular version.

I do not think that generating default.param files and distributing them is a good Idea. I do not distribute them on https://discuss.ardupilot.org/t/how-to-methodically-tune-almost-any-multicopter-using-arducopter-4-4-x/110842 for the reasons you mention.

But I do need to get them in a machine readable format so that I can automate their management. This is the reason for this script.

[andyp1per]

I agree with @IamPete1 about providing defaults files - but this does not do that - it just makes it easier to get at them which seems ok to me

[amilcarlucas]

All checks pass

[amilcarlucas]

Tests pass.