Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AP_RangeFinder: alphabetise type parm docs #28711

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

rmackay9
Copy link
Contributor

@rmackay9 rmackay9 commented Nov 23, 2024

This sorts the list of rangefinders alphabetically which I hope will make it easier for users to find which one they want in the long list. The only imperfection is perhaps 100:SITL appearing near the bottom instead of right at the bottom.

If people don't like this I could manually re-order them.

Before After
0:None 0:None
1:Analog 42:Ainstein_LR_D1
2:MaxbotixI2C 1:Analog
3:LidarLite-I2C 6:BBB-PRU
5:PWM 9:Bebop
6:BBB-PRU 34:Benewake_CAN
7:LightWareI2C 20:Benewake-Serial
8:LightWareSerial 19:BenewakeTF02
9:Bebop 27:BenewakeTF03
10:MAVLink 25:BenewakeTFminiPlus-I2C
11:USD1_Serial 23:BlueRoboticsPing
12:LeddarOne 24:DroneCAN
13:MaxbotixSerial 31:GYUS42v2
14:TeraRangerI2C 30:HC-SR04
15:LidarLiteV3-I2C 41:JRE_Serial
16:VL53L0X or VL53L1X 26:LanbaoPSK-CM8JL65-CC5
17:NMEA 12:LeddarOne
18:WASP-LRF 29:LeddarVu8-Serial
19:BenewakeTF02 21:LidarLightV3HP
20:Benewake-Serial 3:LidarLite-I2C
21:LidarLightV3HP 15:LidarLiteV3-I2C
22:PWM 7:LightWareI2C
23:BlueRoboticsPing 8:LightWareSerial
24:DroneCAN 36:Lua_Scripting
25:BenewakeTFminiPlus-I2C 10:MAVLink
26:LanbaoPSK-CM8JL65-CC5 2:MaxbotixI2C
27:BenewakeTF03 13:MaxbotixSerial
28:VL53L1X-ShortRange 32:MSP
29:LeddarVu8-Serial 17:NMEA
30:HC-SR04 37:NoopLoop_TOFSense
31:GYUS42v2 38:NoopLoop_TOFSense_CAN
32:MSP 40:NoopLoop_TOFSenseF_I2C
33:USD1_CAN 39:NRA24_CAN
34:Benewake_CAN 5:PWM
35:TeraRangerSerial 22:PWM
36:Lua_Scripting 43:RDS02UF
37:NoopLoop_TOFSense 100:SITL
38:NoopLoop_TOFSense_CAN 14:TeraRangerI2C
39:NRA24_CAN 35:TeraRangerSerial
40:NoopLoop_TOFSenseF_I2C 33:USD1_CAN
41:JRE_Serial 11:USD1_Serial
42:Ainstein_LR_D1 16:VL53L0X or VL53L1X
43:RDS02UF 28:VL53L1X-ShortRange
100:SITL 18:WASP-LRF

@rmackay9 rmackay9 added the Docs label Nov 23, 2024
@peterbarker
Copy link
Contributor

@Hwurzburg your thoughts?

@timtuxworth
Copy link
Contributor

I think this is great!

@Hwurzburg
Copy link
Collaborator

LGTM...I like how the list is still numerical in metadata, but presented alphabetically

@amilcarlucas
Copy link
Contributor

Looks like I need to add support for that attribute to the ArduPilot Methodic Configurator then.

@peterbarker peterbarker merged commit 4b040ee into ArduPilot:master Nov 25, 2024
99 checks passed
@rmackay9 rmackay9 deleted the rangefinder-doc-alphabetise branch November 25, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants