adjtomo / seisflows

An automated workflow tool for full waveform inversion and adjoint tomography
http://seisflows.readthedocs.org
BSD 2-Clause "Simplified" License
172 stars 122 forks source link

Fix Seismic Unix (SU) format support and bugfix default preprocessing #174

Open bch0w opened 11 months ago

bch0w commented 11 months ago

Based on discussion raised by @Ben-J-Eppinger in #173, the original version of SeisFlows provided support for the Seismic Unix (SU) format and provided useful preprocessing functions related to exploration-scale seismic problems, such as arrival muting, normalization etc.

In the course of the re-write, these capabilities were brought over but not tested extensively or at all, leading to a broken implementation of these features. Many of the functions and logic loops still exist but will need some bugfixing to get working again.

Tasks for this PR include:

aakash10gupta commented 3 months ago

I think I encountered a bug in the default preprocessing module. When I set the flag components: 'Z' in the parameters.yaml file, even then all the components get used for computing adjoint sources and the subsequent analysis instead of just the Z component.