Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Separate the nav climb rate settings for fw and mc #9424

Merged
merged 3 commits into from
Dec 28, 2023
Merged

Separate the nav climb rate settings for fw and mc #9424

merged 3 commits into from
Dec 28, 2023

Conversation

shota3527
Copy link
Contributor

@shota3527 shota3527 commented Oct 29, 2023 •
edited
Loading

  • Separate the nav climb rate settings for FW and MC
  • No max_fw_auto_climb_rate since it is not used for FW
  • increased default max_fw_manual_climb_rate from 200 to 300, BTW on default value, sin(nav_fw_dive_angle or nav_fw_climb_angle)*54km/h = 3.88 or 5.13 m/s

sim test with VTOL model ok
requires configurator

@shota3527 shota3527 mentioned this pull request Oct 29, 2023
Merged
@DzikuVx DzikuVx added this to the 7.1 milestone Oct 29, 2023
@DzikuVx DzikuVx changed the base branch from master to release_7.1.0 December 28, 2023 10:43
@DzikuVx DzikuVx modified the milestones: 7.1, 8.0 Dec 28, 2023
@DzikuVx
Copy link
Member

DzikuVx commented Dec 28, 2023

I just relalized this is a breaking change and has to go to INAV 8

@DzikuVx DzikuVx changed the base branch from release_7.1.0 to master December 28, 2023 11:07
@DzikuVx DzikuVx merged commit af0b3f4 into iNavFlight:master Dec 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@JulioCesarMatias JulioCesarMatias JulioCesarMatias approved these changes

Assignees
No one assigned
Labels
Requires Configurator
Projects
None yet
Milestone
8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@shota3527 @DzikuVx @JulioCesarMatias @breadoven