Do away with alphanumeric order testing and switch to numbers that correlate with the wikl

iampete · January 10, 2024, 11:50pm

Yeah, its generated from the AP motors example,

github.com/ArduPilot/ardupilot

AP_Motors: example: add ability to dump all matrix motor layouts in JSON format

ArduPilot:master ← IamPete1:motor_layout_json

opened 10:35PM - 18 May 23 UTC

IamPete1

+90 -6

This makes a json file such as this: [MotorsJSON.txt](https://github.com/Ard…uPilot/ardupilot/files/11511969/MotorsJSON.txt) This allows tools to give extra info on motor test pages or frame selection, only matrix types currently but we could others manualy. A example wiki page from @yuri-rage https://discuss.ardupilot.org/t/copter-wiki-motor-diagram-overhaul/101046/24?u=iampete My Mission planner attempt: https://github.com/ArduPilot/MissionPlanner/pull/3043 To get json just run example and dump to file, `./build/linux/examples/AP_Motors_test p > test.json` This is a snippet: ``` { "Version": "AP_Motors library test ver 1.1", "layouts": [ { "Class": 1, "ClassName": "QUAD", "Type": 0, "TypeName": "PLUS", "motors": [ { "Number": 1, "TestOrder": 2, "Rotation": "CCW", "Roll": -0.5000, "Pitch": -0.0000 }, { "Number": 2, "TestOrder": 4, "Rotation": "CCW", "Roll": 0.5000, "Pitch": -0.0000 }, { "Number": 3, "TestOrder": 1, "Rotation": "CW", "Roll": -0.0000, "Pitch": 0.5000 }, { "Number": 4, "TestOrder": 3, "Rotation": "CW", "Roll": 0.0000, "Pitch": -0.5000 } ] }, ] } ```

As far are the flight code is concerned it makes no difference which is on top, so it does not have that information either. So we can’t get that from source, but the convention in the diagrams is that the motor test index top first then bottom. A is top B is bottom ect, so you could apply that convention in the code, for co-located motors. @Yuri_Rage has a great example of generating diagrams from the same file. Copter Wiki Motor Diagram Overhaul? - #24 by Yuri_Rage

Changing motor function at run time works. As you say some functions do require a reboot. Changing frame type eg from X to + also works fine on copter without reboot (not quadplane).