Closed smu160 closed 2 weeks ago
:warning: Please install the to ensure uploads and comments are reliably processed by Codecov.
Attention: Patch coverage is 99.28058%
with 1 line
in your changes missing coverage. Please review.
Please upload report for BASE (
main@2df2f00
). Learn more about missing BASE report.
Files | Patch % | Lines |
---|---|---|
src/twiddles.rs | 96.87% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Planner should be re-usable so it can be re-used for FFT's of the same size
Add regression tests to make sure
fft_64
/fft_32
gives the same results asfft_64_with_opts_and_plan
/fft_32_with_opts_and_plan
@Shnatsel I also added a
criterion
based benchmark based on these benchmarks. In addition, this new benchmark takes into account thatPlanner32
/Planner64
are now re-usable.I am thinking we can use criterion's
BenchmarkGroup
to addrust-fftw3
andRustFFT
for comparison. Ideally, this would replace the current benchmarks.