Skip to content

Autolevel flight mode and pitch trim option#6564

Merged
DzikuVx merged 15 commits intomasterfrom
dzikuvx-autolevel
May 15, 2021
Merged

Autolevel flight mode and pitch trim option#6564
DzikuVx merged 15 commits intomasterfrom
dzikuvx-autolevel

Conversation

@DzikuVx
Copy link
Copy Markdown
Member

@DzikuVx DzikuVx commented Feb 3, 2021

No description provided.

@DzikuVx DzikuVx marked this pull request as ready for review February 19, 2021 08:14
@DzikuVx DzikuVx added this to the 2.7 milestone Feb 19, 2021
@avsaase
Copy link
Copy Markdown
Member

avsaase commented Mar 19, 2021

Hi @DzikuVx, I tried this out this afternoon but the airplane kept losing altitude.

PICT0005_2.mp4

The FC in this plane is mounted pitching forward a bit, so I have set align_board_pitch = -60 to make it fly level in 'normal' angle mode. This also makes the configurator show 0 degrees pitch when the model sits flat on the desk. I did not set fw_level_pitch_trim.

AUTOLEVEL seemed to target -6 degrees pitch down, and set fw_level_pitch_trim = 0.022 as if it thought this was the correct pitch angle. Unfortunately, I do not have a black-box log. Did I misconfigure something or is it a bug?

@DzikuVx
Copy link
Copy Markdown
Member Author

DzikuVx commented Mar 21, 2021

@avsaase the point of this change is not to use align_board_pitch but only fw_level_pitch_trim. From what I see in the DVR, code was not active most of the time. No BB log I can only assume it was due to pitch stick in use. See DEBUG [2] which show currect vertical velocity. It's stabilizing close to 0 but then jumps to positive values when probably you adjust pitch.

@b14ckyy
Copy link
Copy Markdown
Collaborator

b14ckyy commented May 12, 2021

Hey mate. Can you also add the fw_level_pitch_trim to the adjustments and adjustable OSD Items please? Would be beneficial to have it on screen and manually adjustable if on some builds the sensor data are not accurate enough or a craft is not set up with angle/horizon mode.

@DzikuVx DzikuVx merged commit c3a7f72 into master May 15, 2021
@DzikuVx DzikuVx deleted the dzikuvx-autolevel branch May 15, 2021 06:40
@avsaase avsaase added the Release Notes Add this when a PR needs to be mentioned in the release notes label May 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release Notes Add this when a PR needs to be mentioned in the release notes Testing Required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants