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

Offboard reserve mode ID #28939

Merged
merged 6 commits into from
Dec 26, 2024
Merged

Offboard reserve mode ID #28939

merged 6 commits into from
Dec 26, 2024

Conversation

Ryanf55
Copy link
Collaborator

@Ryanf55 Ryanf55 commented Dec 24, 2024

Reserve mode 30 for offboard so it's the same on all vehicles.

@rmackay9
Copy link
Contributor

Thanks for this. I'd vote for using mode 29 or 30. 29 is an odd number but that's OK.

@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Dec 24, 2024

Thanks for this. I'd vote for using mode 29 or 30. 29 is an odd number but that's OK.

I'll do 30!

@timtuxworth
Copy link
Contributor

Does this have to line up with @IamPete1 's work on "scripted modes", where it will be possible to create new modes in Lua script that inherit from Guided?

@Ryanf55
Copy link
Collaborator Author

Ryanf55 commented Dec 24, 2024

Does this have to line up with @IamPete1 's work on "scripted modes", where it will be possible to create new modes in Lua script that inherit from Guided?

Yea that's the idea. We need to work together on a long term design, this is just a reservation if we choose to stop using guided for offboard on plane, and make it consistent between all the vehicles.

Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@tridge tridge merged commit 21c17bd into ArduPilot:master Dec 26, 2024
97 checks passed
@Ryanf55 Ryanf55 deleted the offboard-reserve-id branch December 26, 2024 21:36
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.

4 participants