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

Add RC option for flightmode pause/resume #26748

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

peterbarker
Copy link
Contributor

Tested in SITL with both guided/auto on Copter.

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 for this, we surely need to update some parameter docs..

I wonder if it should be FLIGHTMODE_PAUSE or maybe just Pause? It doesn't work in all modes so we might want to display a failure to the user.

@peterbarker peterbarker added the WikiNeeded needs wiki update label Apr 10, 2024
@peterbarker
Copy link
Contributor Author

LOITER> Mode LOITER
rc height 25
11 2000
LOITER> AP: Flight Mode Pause failed
height 14

@peterbarker
Copy link
Contributor Author

Thanks for this, we surely need to update some parameter docs..

Whoops. Added the parameter docs.

I wonder if it should be FLIGHTMODE_PAUSE or maybe just Pause? It doesn't work in all modes so we might want to display a failure to the user.

Now displays a failure to the user if they try to use it and the mode says no.

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.

LGTM.

I think that Rover could probably support pausing now too now that it uses SCurves. Just musing though and not in any way trying to add to your work load because nobody's actually asked for it yet.

@peterbarker
Copy link
Contributor Author

I think that Rover could probably support pausing now too now that it uses SCurves. Just musing though and not in any way trying to add to your work load because nobody's actually asked for it yet.

@mustafa-gokce did offer an implementation: #19552

@rmackay9
Copy link
Contributor

@peterbarker,

OK, thanks. That implementation is OK but we should do it the same way we've done it for Copter which works better especially when the vehicle is in the middle of a corner.

@peterbarker peterbarker merged commit 18850fd into ArduPilot:master Apr 11, 2024
91 checks passed
@peterbarker peterbarker deleted the pr/rc-pause-resume branch April 11, 2024 02:25
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.

2 participants