Skip to content

FFT: Fix one-d mode in 2D & 3D builds#5225

Merged
WeiqunZhang merged 3 commits intoAMReX-Codes:developmentfrom
WeiqunZhang:fft_batch_1d_2d
Mar 25, 2026
Merged

FFT: Fix one-d mode in 2D & 3D builds#5225
WeiqunZhang merged 3 commits intoAMReX-Codes:developmentfrom
WeiqunZhang:fft_batch_1d_2d

Conversation

@WeiqunZhang
Copy link
Copy Markdown
Member

@WeiqunZhang WeiqunZhang commented Mar 25, 2026

  • Previously it did not handle the single process case correctly.

  • Fix twod_mode=false && oned_mode=true in 3D. Previously, we only support twod_mode=true && oned_mode=true.

  • Fix scaling factor for the one-mode.

  • Add tests.

* Previously it did not handle the single process case correctly.

* Fix scaling factor for the one-mode in 2d builds.

* Add tests.
@WeiqunZhang WeiqunZhang requested review from ajnonaka and atmyers March 25, 2026 02:02
@WeiqunZhang
Copy link
Copy Markdown
Member Author

/run-hpsf-gitlab-ci

@github-actions
Copy link
Copy Markdown

GitLab CI has started at https://gitlab.spack.io/amrex/amrex/-/pipelines/1491596.

@amrex-gitlab-ci-reporter
Copy link
Copy Markdown

GitLab CI 1491596 finished with status: failed. See details at https://gitlab.spack.io/amrex/amrex/-/pipelines/1491596.

@WeiqunZhang
Copy link
Copy Markdown
Member Author

The GitLab CI failure was a machine issue.

Copy link
Copy Markdown
Contributor

@ajnonaka ajnonaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DIM=2 builds work with oned_mode (serial and parallel)
DIM=3 builds work with twod_mode (serial and parallel)

@WeiqunZhang WeiqunZhang changed the title FFT: Fix one-d mode in 2D builds FFT: Fix one-d mode in 2D & 3D builds Mar 25, 2026
@WeiqunZhang
Copy link
Copy Markdown
Member Author

/run-hpsf-gitlab-ci

@github-actions
Copy link
Copy Markdown

GitLab CI has started at https://gitlab.spack.io/amrex/amrex/-/pipelines/1492836.

@amrex-gitlab-ci-reporter
Copy link
Copy Markdown

GitLab CI 1492836 finished with status: failed. See details at https://gitlab.spack.io/amrex/amrex/-/pipelines/1492836.

@WeiqunZhang WeiqunZhang enabled auto-merge (squash) March 25, 2026 22:05
@WeiqunZhang WeiqunZhang merged commit 61e8aea into AMReX-Codes:development Mar 25, 2026
74 checks passed
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.

2 participants