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

getFftData only works for 2^k or 2^extended_k right now #11

Open
jonathanpwang opened this issue Oct 27, 2023 · 0 comments
Open

getFftData only works for 2^k or 2^extended_k right now #11

jonathanpwang opened this issue Oct 27, 2023 · 0 comments

Comments

@jonathanpwang
Copy link

If you want to use #9 (fft mem optimization) with the recursive FFT, you need to do FFT for lengths between 2^k and 2^extended_k. Right now it will give you erroneous FFT data in between.

I fixed this in my dev branch by just making domain.fft_data into a HashMap since usually k..=extended_k is not that big.

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

No branches or pull requests

1 participant