Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvements to Kirchhoff operator #439

Open
2 of 6 tasks
mrava87 opened this issue Sep 24, 2022 · 3 comments
Open
2 of 6 tasks

Improvements to Kirchhoff operator #439

mrava87 opened this issue Sep 24, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@mrava87
Copy link
Collaborator

mrava87 commented Sep 24, 2022

Motivation

The pylops.waveeqprocessing.Kirchhoff operator has gone through a major restyling in #431 mostly aimed at including dynamic capabilities. From the discussions in the #431 and #418 we have identified the following areas that may need further improvement. Moreover some of the new features in such PR are so far only available for the 2D implementation.

Features to be implemented

  • Add 2.5D wavelet filtering in _wavelet_reshaping. Current roadblock is represented by the unclear definition of σ_S and σ_R in equation 2.3.7 of Safron2018 (neither in the Bleistein and Gray, 2002 paper).
  • Devise a new definition of aperture not simply based on the offset/depth ratio to be used for heterogeneous velocity models. Currently we raise a warning when a user decides to activate such aperture limitation
  • Finalize discussion about the role of snell, and if this is actually needed at all.
  • Finalize discussion about angle scaling and introduce cospower as discussed in the PR
  • Implement angle scaling with normal (instead of vertical) when anglerefl is provided
  • Implement all features when dynamic=True in 3D
@mrava87 mrava87 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Sep 24, 2022
@mrava87
Copy link
Collaborator Author

mrava87 commented Oct 8, 2022

For more details on the above points see #431 (comment)

@dikwickley
Copy link
Contributor

hello @mrava87 @cako
i would like to work on this issue. can you please assign it to me?

@mrava87
Copy link
Collaborator Author

mrava87 commented Jan 25, 2023

Sure! get in touch with us (@cako) on slack so we can discuss better maybe where it is better to start :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants