Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

max_throttle adjustments #9915

Merged
merged 2 commits into from
Apr 18, 2024
Merged

max_throttle adjustments #9915

merged 2 commits into from
Apr 18, 2024

Conversation

DzikuVx
Copy link
Member

@DzikuVx DzikuVx commented Apr 9, 2024

This PR removes max_throtlle setting and replaces it with hardcoded:

  • 1850 for rovers and boats
  • 2000 for everything else

Bear in mind, this does not change max power output of motors! This is scaling related only and should help with readability of some settings and blacbox logs. DSHOT outputs is not changed!

Standard/OneShot/Multishot ESC have to be recalibrated for INAV 8

DSHOT users are not affected

@DzikuVx
Drop max_throttle settings and set it hardcoded to 1850 for
3f9d10f 
rovers and boats and 2000 for multicopters and fixedwings.
@DzikuVx DzikuVx added the Release Notes Add this when a PR needs to be mentioned in the release notes label Apr 9, 2024
@DzikuVx DzikuVx added this to the 8.0 milestone Apr 9, 2024
@DzikuVx DzikuVx merged commit b323afb into master Apr 18, 2024
16 checks passed
@DzikuVx DzikuVx deleted the dzikuvx-adjust-max-throttle-settings branch April 18, 2024 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Milestone
8.0
Development

Successfully merging this pull request may close these issues.
1 participant
@DzikuVx