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

GCS_MAVLink: allow uploading GCS to cancel upload using mission_clear_all #27854

Conversation

peterbarker
Copy link
Contributor

@peterbarker peterbarker commented Aug 15, 2024

these would appear to be identical operations. clear differs from a 0-item-upload in the "clear" is only allowed disarmed, and resets a whole bunch of mission state (e.g. the loaded item).

Replaces #15669 . Like the PR it replaces, this allows a GCS to cancel an upload with a clear-all command as if they'd issues a mission-upload with zero items.

@peterbarker peterbarker force-pushed the pr/missionitemprotocol-clear-all-is-zero-item-upload branch from 0e1077f to 5c6a2b2 Compare August 19, 2024 10:53
@peterbarker peterbarker changed the title GCS_MAVLink: treat clear-all as zero-item-upload GCS_MAVLink: allow uploading GCS to cancel upload using mission_clear_all Aug 19, 2024
Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

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

just the name as cancel_upload, thanks

libraries/GCS_MAVLink/MissionItemProtocol.cpp Outdated Show resolved Hide resolved
in the same way that we allow a link to re-start an upload by sending mission_count, allow a GCS to clear a mission and cancel current upload if it was the one doing the transfer
@peterbarker peterbarker force-pushed the pr/missionitemprotocol-clear-all-is-zero-item-upload branch from 5c6a2b2 to 5e2d8b1 Compare August 20, 2024 00:42
@peterbarker peterbarker merged commit ebc8a71 into ArduPilot:master Aug 20, 2024
93 checks passed
@peterbarker peterbarker deleted the pr/missionitemprotocol-clear-all-is-zero-item-upload branch August 20, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants