Skip to content

Improve FFT interpolation (2/2)#117

Open
unalmis wants to merge 21 commits intof0uriest:mainfrom
unalmis:ku/fft
Open

Improve FFT interpolation (2/2)#117
unalmis wants to merge 21 commits intof0uriest:mainfrom
unalmis:ku/fft

Conversation

@unalmis
Copy link

@unalmis unalmis commented Aug 17, 2025

This is the second of two pull requests to improve the FFT interpolation.

The algorithm implemented in #116 (which improves on the implementation in the main branch without any drawbacks), is ideal for interpolation from spectral to real space. For example, given the coefficients of a 2D Fourier series, the optimal way to evaluate that series at uniform points is to compute two sets of 1-dimensional FFTs as done in #116.

When interpolating from real space to real space in 2D, there are four sets of 1-dimensional FFTs to be computed. In some applications, it is of interest to interpolate to a rotated grid which has less density than the original aligned grid. When interpolating to such a grid with less density, it is always better to perform the FFTs such that each dimension is interpolated from real space to real space iteratively.

…ncy spectrum

with negligible additional computation.
…transform

is now applied after (before) the first transform is completed.
This reduces the size of the problem, so the computation is less expensive.
@unalmis unalmis mentioned this pull request Aug 18, 2025
4 tasks
…transform

is now applied after (before) the first transform is completed.
This reduces the size of the problem, so the computation is less expensive.
@unalmis unalmis marked this pull request as draft August 18, 2025 20:18
@unalmis unalmis marked this pull request as ready for review August 19, 2025 06:39
unalmis added a commit to PlasmaControl/DESC that referenced this pull request Aug 24, 2025
@unalmis unalmis marked this pull request as draft August 24, 2025 21:52
@unalmis unalmis marked this pull request as ready for review August 25, 2025 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant