Skip to content

Add 3D slant stack operators#17

Merged
nmbader merged 7 commits intomasterfrom
mbader/slant-stack-3d
Mar 16, 2026
Merged

Add 3D slant stack operators#17
nmbader merged 7 commits intomasterfrom
mbader/slant-stack-3d

Conversation

@nmbader
Copy link
Copy Markdown
Member

@nmbader nmbader commented Mar 16, 2026

  • Add two 3D slant stack operators, one working in the 3D Fourier domain (analogous to the existing 2D operator) and one working in the original time-space domain (shift and sum).
  • Add 2D slant stack operator working in the time-space domain (shift and sum)
  • Simplify and fix bug in the existing 2D slant stack operator
  • Add unit tests

Note 1: in 3D with mode="depth", the mapping from subsurface offsets to aperture angle depends on the geologic dip.
Note 2: the shift and sum operator is typically slower than the Fourier domain operator. However, it yields result with less artifacts and can serve as a verification test. Moreover, the 3D shift and sum in mode="depth" can accommodate depth-varying geologic dip, which is not possible in the Fourier domain.

Copy link
Copy Markdown
Member

@samtkaplan samtkaplan left a comment

Choose a reason for hiding this comment

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

Looks good. I'm wondering if WaveFD is a bit of a heavy dep to add, but I'm not sure I have a better suggestion at the moment.

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.

nice add

shift/sum allows non-regular locations but I guess we are assuming everything regular

@nmbader
Copy link
Copy Markdown
Member Author

nmbader commented Mar 16, 2026

Looks good. I'm wondering if WaveFD is a bit of a heavy dep to add, but I'm not sure I have a better suggestion at the moment.

I will try a julia native package for shifting instead of WaveFD

@nmbader
Copy link
Copy Markdown
Member Author

nmbader commented Mar 16, 2026

nice add

shift/sum allows non-regular locations but I guess we are assuming everything regular

yes, the shift/sum operator can take either regular offsets in xy or fully irregular offsets

@nmbader nmbader merged commit 7d2981d into master Mar 16, 2026
7 checks passed
Copy link
Copy Markdown
Member Author

@nmbader nmbader left a comment

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

@nmbader nmbader left a comment

Choose a reason for hiding this comment

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

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.

4 participants