Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request makes the generic FFT and DCT routines in FastTransforms available for all AbstractFloats. Some changes to the codeflow are described in #62. An earlier issue was #34.
The code is written such that the more efficient plans of FFTW are still returned when T is one of the fftwNumber types. There is currently no check for this, which is dangerous. I suppose we could check on the type of the plan returned by
plan_fft
, but I'm not sure how to check which method offft
ends up being invoked. I would not be surprised if this pull request breaks things. The implementation remains a lot slower than FFTW of course, and it is also less accurate.My goal was to make it work for
DoubleFloats.jl
types, such asDouble64
. This works now:So, for this example,
Double64
is faster thanBigFloat
by a factor of about 20. It is slower than FFTW by a factor of about 200.