Skip to content

default PID Denom 2 for F411#994

Merged
nerdCopter merged 2 commits intoemuflight:masterfrom
nerdCopter:20240328_modern_pid_denom
Apr 2, 2024
Merged

default PID Denom 2 for F411#994
nerdCopter merged 2 commits intoemuflight:masterfrom
nerdCopter:20240328_modern_pid_denom

Conversation

@nerdCopter
Copy link
Copy Markdown
Member

@nerdCopter nerdCopter commented Mar 28, 2024

@nerdCopter
Copy link
Copy Markdown
Member Author

@Quick-Flash , @BeauBrewski what do you think about this quick-fix for F411 denom?

@nerdCopter
Copy link
Copy Markdown
Member Author

  • tested on CRAZYBEEF4DX (F411) resulted denom 2 and dshot300
  • tested SKYSTARSF405-AIO resulted denom 1 and dshot600
  • tested PYDRODRONEF7 resulted denom 1 and dshot600
  • tested HELIOSPRING resulted denom 1 and dshot600

@Quick-Flash
Copy link
Copy Markdown
Member

No need for dshot 300 as a default. If hardware is setup poorly and but any dshot is required it'll run slower than dshot600 as well. So 600 is a better default imo.

@nerdCopter
Copy link
Copy Markdown
Member Author

nerdCopter commented Mar 28, 2024

okay, thanks, reverted mixer.c.

@nerdCopter nerdCopter changed the title default PID Denom 2 and DSHOT300 for F411 default PID Denom 2 for F411 Mar 28, 2024
@nerdCopter nerdCopter merged commit 1259124 into emuflight:master Apr 2, 2024
@nerdCopter nerdCopter deleted the 20240328_modern_pid_denom branch April 2, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

F411 are often 100% cpu

2 participants