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

gyro_fft: improve peak detection, add start parameter #16048

Merged
merged 1 commit into from
Oct 26, 2020
Merged

gyro_fft: improve peak detection, add start parameter #16048

merged 1 commit into from
Oct 26, 2020

Conversation

dagar
Copy link
Member

@dagar dagar commented Oct 26, 2020 •
edited
Loading

  • add new parameter IMU_GYRO_FFT_EN to start
  • add 75% overlap in buffer to increase FFT update rate
  • space out FFT calls (no more than 1 per cycle)
  • increase IMU_GYRO_FFT_MIN default
  • decrease main stack usage

@dagar
Copy link
Member Author

dagar commented Oct 26, 2020 •
edited
Loading

Working, but cleanly detecting and following peaks still needs improvement. For now I've raised the default minimum frequency (IMU_GYRO_FFT_MIN).

Screen Shot 2020-10-25 at 10 59 56 PM

@dagar
gyro_fft: improve peak detection, add start parameter
fe5733a 
 - space out FFT calls (no more than 1 per cycle)
 - increase IMU_GYRO_FFT_MIN default
@dagar dagar merged commit 614a0ac into master Oct 26, 2020
@dagar dagar deleted the pr-fft_improve branch October 26, 2020 03:48
@bresch
Copy link
Member

bresch commented Oct 26, 2020

This is awesome!

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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dagar @bresch