-
Notifications
You must be signed in to change notification settings - Fork 154
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
replace numpyfft with scipyfft for better performance #892
base: main
Are you sure you want to change the base?
replace numpyfft with scipyfft for better performance #892
Conversation
compared to other avialble options numpy fft and scipy fft seem to have higher accuracy as well |
@Sauravroy34 interesting, thanks. Do you have measurements for the 1D transform we use as well? |
sure , i could not find data for 1 d in internet so i decided of making one
using pyfftw interfaces |
normal scipy fft and numpy fft |
@Sauravroy34 would you mind re-running the benchmark inverting the order of the calls to numpy and scipy? Just as a sanity check |
inverting the calls seems to have reverse effect on plots it might be due to some caching effect i will try to create a uniform benchmark for both of them and post update here |
@matteobachetti in this comment the benchmark was conducted |
i have conducted few test myself here is the notebook https://github.com/Sauravroy34/Benchmark_test_for_scipy_and_numpy |
it seems that scipy.fft is faster than numpy fft
sources
1 ) https://stackoverflow.com/questions/6363154/what-is-the-difference-between-numpy-fft-and-scipy-fftpack
2 ) https://medium.com/@corentin.soubeiran/whos-the-most-efficient-fft-techniques-in-python-boosting-performance-and-understanding-d19b6641d6d8