Skip to content

bugfixes with Fourier slantstack - accelaration of 3D shiftsum - Unit tests#19

Merged
nmbader merged 9 commits intomasterfrom
mbader/bugfix-geologic-dip
Mar 26, 2026
Merged

bugfixes with Fourier slantstack - accelaration of 3D shiftsum - Unit tests#19
nmbader merged 9 commits intomasterfrom
mbader/bugfix-geologic-dip

Conversation

@nmbader
Copy link
Copy Markdown
Member

@nmbader nmbader commented Mar 26, 2026

jop_slantstack updates:

  • Correct the mapping formula in the 3D wavenumber domain in the presence of geologic dip
  • Correct the out-of-bounds wavenumbers check for all Fourier operator (previous implementation was incorrect beyond the aliased wavenumbers)
  • Speed up the ShiftSum operator in 3D when depth variable dip is present by replaced the Shifting operation by direction Interpolation of the appropriate sample
  • Improve multithreaded double loops and reducing local array allocations
  • Robustify unit tests and add new ones

Copy link
Copy Markdown
Member

@jkwashbourne-oss jkwashbourne-oss left a comment

Choose a reason for hiding this comment

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

looks great, thanks for the improvements

@nmbader nmbader merged commit 91d1762 into master Mar 26, 2026
7 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.

3 participants