ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

Precland cleanups

Open amilcarlucas opened this issue 1 year ago • 4 comments

Removes code duplication in SITL Make SITL behave like other backends Small (mostly NFC) cleanups on the Precland that make the future sensor lag estimation PR smaller.

amilcarlucas avatar Feb 15 '24 10:02 amilcarlucas

Binary Name      Text [B]      Data [B]     BSS (B)       Total Flash Change [B] (%)      Flash Free After PR (B)
---------------  ------------  -----------  ------------  ----------------------------  -------------------------
ardurover        4 (+0.0002%)  0 (0.0000%)  4 (+0.0015%)  4 (+0.0002%)                                     337332
blimp            0 (0.0000%)   0 (0.0000%)  0 (0.0000%)   0 (0.0000%)                                      623072
arducopter       0 (0.0000%)   0 (0.0000%)  0 (0.0000%)   0 (0.0000%)                                      192988
arduplane        0 (0.0000%)   0 (0.0000%)  0 (0.0000%)   0 (0.0000%)                                      201296
ardusub          0 (0.0000%)   0 (0.0000%)  0 (0.0000%)   0 (0.0000%)                                      400608
antennatracker   0 (0.0000%)   0 (0.0000%)  0 (0.0000%)   0 (0.0000%)                                      645948
arducopter-heli  8 (+0.0005%)  0 (0.0000%)  0 (0.0000%)   8 (+0.0004%)                                     186812

amilcarlucas avatar Feb 15 '24 12:02 amilcarlucas

This has been tested in CI

amilcarlucas avatar Feb 19 '24 08:02 amilcarlucas

needs approval from @rishabsingh3003

tridge avatar Feb 21 '24 07:02 tridge

I have three other PRs that build on top of this one.

amilcarlucas avatar Feb 22 '24 10:02 amilcarlucas

@rishabsingh3003 ping

amilcarlucas avatar Feb 29 '24 15:02 amilcarlucas

@amilcarlucas I'd like to delay this till after https://github.com/ArduPilot/ardupilot/pull/26380 is merged as I'd like that in 4.5, and this would make backporting to 4.5.x much harder

tridge avatar Mar 04 '24 23:03 tridge

@tridge it rebased cleanly

amilcarlucas avatar Mar 06 '24 00:03 amilcarlucas