betaflight-configurator icon indicating copy to clipboard operation
betaflight-configurator copied to clipboard

TPA clarification - % and μs

Open nerdCopter opened this issue 1 year ago • 2 comments

  • resolves #3844
  • will need Crowdin translations

image

nerdCopter avatar Mar 20 '24 13:03 nerdCopter

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

github-actions[bot] avatar Mar 20 '24 13:03 github-actions[bot]

Incoming pedantry. 🤣

0.65 is not a % value. We need to multiply the number by 100 as well. I'm not sure us makes this more understandable. If one is using Dshot then the breakpoint is not a microsecond value at all but rather a value between 0 and 2000 I think.

ChrisRosser avatar Mar 21 '24 19:03 ChrisRosser

@haslinghuis , should we revert Configurator? i'm fine with such.

correct, but humans will read 0.65 as 65% quite easily. pardon any break-point mishaps :frowning_face:

nerdCopter avatar Mar 21 '24 19:03 nerdCopter

tpa breakpoint is a pwm value between 750 and 2250 tpa_rate is between 0 and 1

Take 2: https://github.com/betaflight/betaflight-configurator/pull/3851 :smiley:

haslinghuis avatar Mar 21 '24 19:03 haslinghuis

in fact a good change

HThuren avatar Mar 21 '24 20:03 HThuren