Skip to content

Copter/TradHeli: PILOT parameters renamed and converted to meters#32184

Merged
rmackay9 merged 9 commits intoArduPilot:masterfrom
rmackay9:copter47-pilot-param-conv
Mar 4, 2026
Merged

Copter/TradHeli: PILOT parameters renamed and converted to meters#32184
rmackay9 merged 9 commits intoArduPilot:masterfrom
rmackay9:copter47-pilot-param-conv

Conversation

@rmackay9
Copy link
Copy Markdown
Contributor

This PR is a follow-up to #32169 and helps address issue #31562

These parameters have been renamed/converted:

PILOT_ACCEL_Z (cm/s) -> PILOT_ACC_Z (m/s)
PILOT_SPEED_UP (cm/s) -> PILOT_SPD_UP (m/s)
PILOT_SPEED_DN (cm/s) -> PILOT_SPD_DN (m/s)
PILOT_TKOFF_ALT (cm) -> PILOT_TKO_ALT_M (meters)

This has been lightly tested in SITL but only to confirm that the parameter conversion is working

pilot-params-before-vs-after

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 rmackay9 force-pushed the copter47-pilot-param-conv branch 2 times, most recently from f9f0b52 to 6bfc7dc Compare February 13, 2026 02:45
@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Feb 13, 2026
@rmackay9 rmackay9 force-pushed the copter47-pilot-param-conv branch from 6bfc7dc to 6833df1 Compare February 14, 2026 21:26
@timtuxworth
Copy link
Copy Markdown
Contributor

More evidence that WPNAV should go WPNAV_ACC WPNAV_SPD etc as they will now line up!

@rmackay9 rmackay9 force-pushed the copter47-pilot-param-conv branch from 6833df1 to fb6f7ae Compare February 17, 2026 06:24
@rmackay9
Copy link
Copy Markdown
Contributor Author

merge conflicts resolved and rebased on master

@rmackay9
Copy link
Copy Markdown
Contributor Author

rmackay9 commented Mar 4, 2026

I've lightly tested this in SITL and it seems to be working so I'll go ahead and merge it, thanks!

@rmackay9 rmackay9 merged commit 28ac644 into ArduPilot:master Mar 4, 2026
105 checks passed
@rmackay9 rmackay9 deleted the copter47-pilot-param-conv branch March 4, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants