search

JuliaDSP / FourierTransforms.jl

Fourier transforms written in Julia
MIT License
20 stars 2 forks source link

Start generalizing to fields other than Complex #7

Closed Keno closed 4 years ago

Keno commented 5 years ago

Also add a symbolic testing utility that traces through the fft computation and makes sure that it's symbolically equal to the native fft evluation order (quite useful also for tracing through what the various steps of the fft are doing).

Keno commented 5 years ago

Fixed the test failure.

codecov-io commented 5 years ago

Codecov Report

Merging #7 into master will increase coverage by 1.91%. The diff coverage is 89.36%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #7      +/-   ##
=========================================
+ Coverage   66.98%   68.9%   +1.91%     
=========================================
  Files           3       3              
  Lines         312     328      +16     
=========================================
+ Hits          209     226      +17     
+ Misses        103     102       -1
Impacted Files Coverage Δ
src/ctfft.jl 78.12% <89.36%> (+1.87%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update dde2178...9611828. Read the comment docs.

Keno commented 4 years ago

Since I'm the only user of this package for now, I'll merge this and we can clean up on master later.