Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #105 +/- ##
==========================================
+ Coverage 98.18% 98.80% +0.62%
==========================================
Files 9 10 +1
Lines 1871 2926 +1055
==========================================
+ Hits 1837 2891 +1054
- Misses 34 35 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Ideally I'd like to merge #95 so that we have in-place interleaving/deinterleaving and then make this operate in-place too. Deinterleaving is trivial, it's the other step that might need some care when being performed in-place. That doesn't have to happen in this PR but it is something I'd like to tackle before shipping a release. |
Addresses issues #115 `PlannerDit32/PlannerDit64` no longer require a `Direction`. The `fft_*_dit_with_planner` and `fft_*_dit_with_planner_and_opts` entry points (and their interleaved variants) now take `Direction` as an explicit argument, positioned after the buffers and before the planner. The same planner instance can now drive, both, forward and inverse transforms, which removes the need to maintain two planners for round-trip workflows.
c2r already implies inverse FFT
|
|
Super curious found this by chance. If there is any interest, you could benchmark against https://github.com/awxkee/zaft it's a bit more optimized for aarch64 and needs to be compiled with nightly with the fcma feature for that, but for x86_64 it should work too |
4 participants
No description provided.