search

jipolanco / PencilFFTs.jl

Fast Fourier transforms of MPI-distributed Julia arrays
https://jipolanco.github.io/PencilFFTs.jl/dev/
MIT License
77 stars 7 forks source link

Improve support for BRFFT (complex-to-real FFT) #33

Closed jipolanco closed 2 years ago

jipolanco commented 3 years ago

As described in the AbstractFFTs docs, the output size of a complex-to-real transform may not be fully determined from the input size.

This PR adds an optional even_output parameter to the Transforms.BRFFT constructor, which allows to fully determine the output size given the input and this additional information.

codecov[bot] commented 3 years ago

Codecov Report

Merging #33 (68588ec) into master (2d3e262) will decrease coverage by 0.20%. The diff coverage is 96.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   98.52%   98.31%   -0.21%     
==========================================
  Files           9        9              
  Lines         407      416       +9     
==========================================
+ Hits          401      409       +8     
- Misses          6        7       +1
Impacted Files Coverage Δ
src/Transforms/r2c.jl 92.85% <92.30%> (-7.15%) :arrow_down:
src/Transforms/Transforms.jl 94.44% <100.00%> (+2.13%) :arrow_up:
src/Transforms/c2c.jl 100.00% <100.00%> (+4.34%) :arrow_up:
src/Transforms/no_transform.jl 100.00% <100.00%> (ø)
src/Transforms/r2r.jl 100.00% <100.00%> (ø)
src/global_params.jl 100.00% <100.00%> (ø)
src/plans.jl 97.03% <100.00%> (-0.03%) :arrow_down:

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 2d3e262...68588ec. Read the comment docs.