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.

Sign up for GitHub

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

Jump to bottom

Copter, AP_Mission: Send progress for individual mission items (take 2) #27779

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from
Draft

Copter, AP_Mission: Send progress for individual mission items (take 2) #27779

wants to merge 13 commits into from
+250 −7

Conversation

nexton-winjeel
Copy link
Contributor

Replaces #25249

This PR:

  • adds the framework in AP_Mission for populating and transmitting the progress fields of a new NAV_CONTROLLER_PROGRESS message; and
  • updates Copter to send this information for a variety of mission items.

Requires: ArduPilot/mavlink#366

@nexton-winjeel
GCS_MAVLink: Add guard for NAV_CONTROLLER_PROGRESS message
05e8466
@nexton-winjeel
AP_Mission: Add functions to set the progress of the current mission …
dfd0688 
…item
@nexton-winjeel
Copter: Populate item progress for loiter time
457d5f7
@nexton-winjeel
Copter: Populate item progress for waypoints
37e72bb
@nexton-winjeel
Copter: Populate item progress for loiters
6e5193e
@nexton-winjeel
Copter: Populate item progress for spline waypoints
98484a7
@nexton-winjeel
Copter: Populate item progress for nav delays
b9f6335
@nexton-winjeel
Copter: Populate item progress for loiter to alt
d3a659d
@nexton-winjeel
GCS_MAVLink: Send NAV_CONTROLLER_PROGRESS for mission items
e332fac
@nexton-winjeel
Copter: Add NAV_CONTROLLER_PROGRESS to Extended Status stream
ca8711b
@nexton-winjeel
AP_HAL_ChibiOS: Disable NAV_CONTROLLER_PROGRESS msg in minimize_commo…
b5c78e5 
…n.inc
@nexton-winjeel
Tools: Add NAV_CONTROLLER_PROGRESS guard to build_options.py
789200e
@nexton-winjeel
Tools: Add check for NAV_CONTROLLER_PROGRESS to extract_features.py
9a0769f
@nexton-winjeel
Copy link
Contributor Author

Example of what the GCS side looks like can be seen here: mavlink/mavlink#2029 (comment)

@nexton-winjeel
Copy link
Contributor Author

Uses ~1232 bytes of flash (as tested on CubeOrange):

          Target          Text (B)  Data (B)  BSS (B)  Total Flash Used (B)  Free Flash (B)  External Flash Used (B)
          ----------------------------------------------------------------------------------------------------------
enabled:  bin/arducopter   1841556      3920   258372               1845476          120588  Not Applicable 
disabled: bin/arducopter   1840324      3920   258372               1844244          121820  Not Applicable

@peterbarker peterbarker mentioned this pull request Sep 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nexton-winjeel