Skip to content

Conversation

@breadoven
Copy link
Collaborator

@breadoven breadoven commented Dec 21, 2020

PR provides an option to use a loiter climb (spiral) rather than a linear climb during fixed wing RTH when Climb First setting is used. Effectively uses the same method as that used for height change to the RTH home altitude but for the initial climb phase instead.

Climb rate set by max_auto_climb_rate, turn set by nav_fw_loiter_radius.

Default set to OFF.

Tested in flight over several months and works as expected.

Uses loiter climb (spiral) for FW RTH with Climb First option.
This reverts commit b6279d8.
@breadoven
Copy link
Collaborator Author

Don't know what the issue is with settings.md. Seems to be comparing with 2.6.0 version not the current master version ?

@Jetrell
Copy link

Jetrell commented Dec 21, 2020

This sounds like a good idea. Especially if we are flying in a valley or canyon.

What do you think of having its selection under the Modes or Adjustment tab? That way the RTH climb type could be changed on the fly.

@avsaase
Copy link
Member

avsaase commented Jan 16, 2021

Would it not make more sense to have a option nav_rth_climb_mode with three options: CLIMB_FIRST, TURN_FIRST and SPIRAL_CLIMB. Only thing to keep in mind is that SPIRAL CLIMB should not be possible on multirotors.

@breadoven
Copy link
Collaborator Author

I did think of a 3 choice option but wasn't sure it made sense for multirotors. I'll add it to my test build anyway so it can be added to this PR if required.

@breadoven
Copy link
Collaborator Author

Kept nav_rth_climb_first setting but with 3 options, OFF, ON, SPIRAL. Avoids issues with setting compatibility.

@avsaase avsaase added this to the 3.0 milestone Apr 12, 2021
@avsaase avsaase added the Release Notes Add this when a PR needs to be mentioned in the release notes label Apr 12, 2021
@avsaase avsaase merged commit 455456f into iNavFlight:master Apr 12, 2021
@breadoven breadoven deleted the abo_RTH_spiral_climb branch April 13, 2021 08:43
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants