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

Move handling of GCS terrain requests up into common GCS library #21150

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Move handling of GCS terrain requests up into common GCS library #21150

wants to merge 7 commits into from

Conversation

peterbarker
Copy link
Contributor

No need for the vehicles to do this themselves any more.

Costs a few bytes with the nullptr checks.

@peterbarker peterbarker force-pushed the pr/terrain-link branch 2 times, most recently from be0024d to 6bf548b Compare July 8, 2022 12:32
@peterbarker peterbarker force-pushed the pr/terrain-link branch 3 times, most recently from adc0789 to 0be99ba Compare July 17, 2022 11:47
@peterbarker peterbarker force-pushed the pr/terrain-link branch 4 times, most recently from 6b60dc2 to 4db3987 Compare August 11, 2022 23:59
@peterbarker
Copy link
Contributor Author

Pixhawk1-1M:

Binary Name      Text [B]         Data [B]     BSS (B)        Total Flash Change [B] (%)      Flash Free After PR (B)
---------------  ---------------  -----------  -------------  ----------------------------  -------------------------
antennatracker   1540 (+0.2051%)  0 (0.0000%)  4 (+0.0020%)   1540 (+0.2048%)                                  278768
arducopter-heli  56 (+0.0055%)    0 (0.0000%)  0 (0.0000%)    56 (+0.0055%)                                     18164
ardurover        2096 (+0.2282%)  0 (0.0000%)  0 (0.0000%)    2096 (+0.2279%)                                  110516
arducopter       56 (+0.0056%)    0 (0.0000%)  0 (0.0000%)    56 (+0.0056%)                                     30188
ardusub          32 (+0.0035%)    0 (0.0000%)  0 (0.0000%)    32 (+0.0035%)                                    111376
blimp            1528 (+0.2157%)  0 (0.0000%)  0 (0.0000%)    1528 (+0.2154%)                                  321260
arduplane        68 (+0.0066%)    0 (0.0000%)  -4 (-0.0020%)  68 (+0.0066%)                                       276

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
Peter's ArduPilot 4.7 Queue
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@peterbarker