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

Sort parameters with numerics #29155

Open
Ryanf55 opened this issue Jan 28, 2025 · 5 comments
Open

Sort parameters with numerics #29155

Ryanf55 opened this issue Jan 28, 2025 · 5 comments
Assignees
Labels

Comments

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Jan 28, 2025

Feature request

Is your feature request related to a problem? Please describe.

Docs for params show up in a weird order.

Image

Describe the solution you'd like

see https://github.com/ArduPilot/ardupilot/blob/master/Tools/autotest/logger_metadata/emit_xml.py#L54

Describe alternatives you've considered
Manual ordering
Platform
[ ] All
[ ] AntennaTracker
[ ] Copter
[ ] Plane
[ ] Rover
[ ] Submarine

@Ryanf55 Ryanf55 added the Docs label Jan 28, 2025
@Ryanf55 Ryanf55 self-assigned this Jan 28, 2025
@rmackay9
Copy link
Contributor

Thanks. Also SERVO10_xxx appears before SERVO2_xxx

@LupusTheCanine
Copy link

The issue is that parameters are sorted alphabetically and numbers do not maintain consistent width, we would have to rename all parameters where there are 10 or more instances to always use two digits numbers. IMHO it would be nicer if we could show instantiated parameters as for example SERVONN_*so we don't have to scroll or search through 32 instances of servo parameters.

@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Jan 28, 2025

We discussed in dev call yesterday and the recommended approach is to use a regex to discover numerics. We need to sort by the numerically instead of lexographically, which should solve the less-than-10 more-than-10 issue that the code currently has.

@LupusTheCanine
Copy link

We discussed in dev call yesterday and the recommended approach is to use a regex to discover numerics. We need to sort by the numerically instead of lexographically, which should solve the less-than-10 more-than-10 issue that the code currently has.

Please consider grouping them, this will make searching through parameters so much faster.

@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Jan 29, 2025

I'll see what I can do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants