Skip to content

AC_Avoid: ANGLE_MAX renamed to ANG_MAX and converted to degrees#32186

Merged
rmackay9 merged 3 commits intoArduPilot:masterfrom
rmackay9:copter47-avoid-param-conv
Mar 4, 2026
Merged

AC_Avoid: ANGLE_MAX renamed to ANG_MAX and converted to degrees#32186
rmackay9 merged 3 commits intoArduPilot:masterfrom
rmackay9:copter47-avoid-param-conv

Conversation

@rmackay9
Copy link
Copy Markdown
Contributor

This continues the drip-feed of parameter conversions to SI units. This is a follow-up to PR #32184 and helps address #31562

Only a single parameter is affected by this PR

AVOID_ANGLE_MAX (centi-degrees) -> AVOID_ANG_MAX (degrees)

This has been lightly tested in SITL but only to confirm that the parameter conversion is working. TBH, so few users use the non-GPS avoidance that we should consider removing the conversion in order to save flash

avoid-angle-max-before-vs-after

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Feb 13, 2026
@rmackay9 rmackay9 changed the title AC_Avoid: ANGLE_MAX renamed to ANG_MAX anc converted to degrees AC_Avoid: ANGLE_MAX renamed to ANG_MAX and converted to degrees Feb 14, 2026
@rmackay9 rmackay9 force-pushed the copter47-avoid-param-conv branch from 07316de to 4705071 Compare February 14, 2026 21:20
Comment thread libraries/AC_Avoidance/AC_Avoid.cpp Outdated
Comment thread libraries/AC_Avoidance/AC_Avoid.cpp Outdated
Comment thread libraries/AC_Avoidance/AC_Avoid.cpp Outdated
@rmackay9 rmackay9 force-pushed the copter47-avoid-param-conv branch from 4705071 to 249a517 Compare February 17, 2026 06:29
@rmackay9 rmackay9 force-pushed the copter47-avoid-param-conv branch from 249a517 to aa8ad84 Compare March 4, 2026 05:03
Copy link
Copy Markdown
Contributor

@lthall lthall left a comment

Choose a reason for hiding this comment

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

Line by line looks good.

@rmackay9
Copy link
Copy Markdown
Contributor Author

rmackay9 commented Mar 4, 2026

I've tested this lightly in SITL and it's working so I'll merge once it passes CI, thanks!

@rmackay9 rmackay9 force-pushed the copter47-avoid-param-conv branch from dd1e489 to aa8ad84 Compare March 4, 2026 05:23
@rmackay9 rmackay9 merged commit 567eb2a into ArduPilot:master Mar 4, 2026
109 of 203 checks passed
@rmackay9 rmackay9 deleted the copter47-avoid-param-conv branch March 4, 2026 10:11
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.

5 participants