search

JuliaApproximation / GenericFFT.jl

A package for computing the FFT with arbitrary floating point numbers
MIT License
15 stars 4 forks source link

Restore interlace #41

Closed dlfivefifty closed 11 months ago

codecov[bot] commented 11 months ago

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (1e08ab9) 82.50% compared to head (a9f7efb) 77.46%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #41 +/- ## ========================================== - Coverage 82.50% 77.46% -5.04% ========================================== Files 2 2 Lines 200 213 +13 ========================================== Hits 165 165 - Misses 35 48 +13 ``` | [Files](https://app.codecov.io/gh/JuliaApproximation/GenericFFT.jl/pull/41?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaApproximation) | Coverage Δ | | |---|---|---| | [src/fft.jl](https://app.codecov.io/gh/JuliaApproximation/GenericFFT.jl/pull/41?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaApproximation#diff-c3JjL2ZmdC5qbA==) | `77.77% <0.00%> (-5.22%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jishnub commented 11 months ago

Should FastTransforms copy over the interlace definition instead of relying on this package?

dlfivefifty commented 11 months ago

Probably. We can remove it in the next breaking version