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.

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

Add angle mode center sensitivity #3910

Merged
merged 2 commits into from
Apr 27, 2024
Merged

Conversation

haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Apr 26, 2024

image

Thanks to @ctzsnooze for functional design.

  • Needs clarification in documentation.

Copy link

netlify bot commented Apr 26, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 2847414
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/662c509c49f39a000881a29f
😎 Deploy Preview https://deploy-preview-3910--origin-betaflight-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

This comment has been minimized.

@haslinghuis haslinghuis added this to the 11.0 milestone Apr 26, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@haslinghuis haslinghuis force-pushed the angle-mode branch 2 times, most recently from 7d8bf57 to 34539a0 Compare April 27, 2024 00:28

This comment has been minimized.

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

Do you want to test this code? Here you have an automated build:
Betaflight-Configurator-Linux
Betaflight-Configurator-Android
Betaflight-Configurator-macOS
Betaflight-Configurator-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

@ctzsnooze
Copy link
Member

Thanks, Mark - this is great! I hope Angle users find it helpful.

@ctzsnooze
Copy link
Member

Important note - the value for 'Angle Center Sensitivity` is effectively in degrees; it is the angle you'd get, in Angle Mode, at max stick position, if the centre sensitivity was held the whole way to full stick travel.

This value allows Angle Mode pilots to get a value that precisely indicates their centre stick sensitivity for Actual or Betaflight Rates; the values are directly comparable between both Rates models.

Note also that centre stick sensitivity in Angle Mode will change if the user modifies their angle_limit value. The centre sticks sensitivity value shown here correctly adapts to angle_limit changes.

Some images showing the numbers you might see, all with the default angle_limit of 60...

Default Actual Rates, showing weak centre sensitivity of only 6.3:
Screen Shot 2024-04-27 at 11 37 37 am

Betaflight Rates, showing default centre sensitivity in Angle Mode of 18 - three times 'twitchier' than the default for Actual Rates...

Screen Shot 2024-04-27 at 11 32 34 am

Actual Rates, modified with values that closely resemble default Betaflight rates, with approximately the same centre sensitivity:

Screen Shot 2024-04-27 at 11 33 30 am

Copy link
Member

@ctzsnooze ctzsnooze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works really well!

@blckmn blckmn merged commit 9e07b12 into betaflight:master Apr 27, 2024
12 checks passed
chmelevskij pushed a commit to chmelevskij/betaflight-configurator that referenced this pull request Apr 27, 2024
@haslinghuis haslinghuis modified the milestones: 11.0, 10.10.0 Apr 27, 2024
@haslinghuis haslinghuis deleted the angle-mode branch April 27, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: COMPLETED
Development

Successfully merging this pull request may close these issues.

3 participants