Improvements to Kirchhoff operator

Aperta
  • Qualcosa non va?
#439 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

@dikwickley ci sta già lavorando.

Dal 25/1/2023.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
15/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
python
Ambito
backend

Direzione di ricerca

Start by reading the discussions in PRs #431 and #418, then inspect pylops.waveeqprocessing.Kirchhoff and _wavelet_reshaping. The issue leaves key definitions unresolved, including the 2.5D wavelet parameters and aperture for heterogeneous velocities. Done means resolving the remaining design questions and implementing the unchecked features, including dynamic features in 3D.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement good first issue help wanted

Motivation

The pylops.waveeqprocessing.Kirchhoff operator has gone through a major restyling in https://github.com/PyLops/pylops/pull/431 mostly aimed at including dynamic capabilities. From the discussions in the https://github.com/PyLops/pylops/pull/431 and https://github.com/PyLops/pylops/pull/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
Lingua principale
Python
Stelle
539
Fork
126
Merge medio
2g 17h
PR unite (30g)
21

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di PyLops/pylops

Tutte le issue di PyLops/pylops

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.