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

MavCmd: alphebetise and fixup rover command list #2905

Merged
merged 7 commits into from Jul 13, 2022

Conversation

rmackay9
Copy link
Contributor

This PR makes a few changes primarily to the Rover command list (there is one change to the Plane list)

  • alphebetise the rover command list. Like Copter's list the nav commands appear at the top with the two most common commands (WAYPOINT and RETURN_TO_LAUNCH) appearing right at the top. Other less popular nav commands are then in alphabetical order. "do" and "conditional" commands are next and are all in alphabetical order.
  • removed unsupported do-set-home
  • removed unsupported do-mount-configure from both rover and plane's lists.
  • added titles to the do-digicam-configure and do-digicam-control messages (for rover)
  • removed unsupported arguments from do-change-speed (for rover)

I have tested this on my local machine by copying the mavcmd.xml and confirming the commands appear as expected

image

@meee1 meee1 merged commit 98b8e58 into ArduPilot:master Jul 13, 2022
@rmackay9
Copy link
Contributor Author

Thanks!

@rmackay9 rmackay9 deleted the rover-mis-reorder branch July 13, 2022 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants