Fix multiple stfs - Githubissues

seismology / mc_kernel

Calculate seismic sensitivity kernels using Axisem

GNU General Public License v3.0

24 stars 8 forks source link

Fix multiple stfs #49

Closed sstaehler closed 7 years ago

sstaehler commented 7 years ago

Problem reported by @kasra-hosseini: Code is crashing when using multiple Source Time Functions. Found that it was due to inconsistencies in filter initialization.

Call filter initialization for each STF
Write out filters separately for each STF
more stability in suppression of high frequencies above AxiSEM dominant period.

Added TRAVIS test run for this problem.

coveralls commented 7 years ago

Coverage increased (+0.2%) to 83.549% when pulling 7f855d025a0e45581e241fbc462c251c8da22a57 on fix_multiple_stfs into d3b7cbd6e0a1d632257a0267a0053e474239de71 on master.

coveralls commented 7 years ago

Coverage increased (+0.2%) to 83.549% when pulling 7f855d025a0e45581e241fbc462c251c8da22a57 on fix_multiple_stfs into d3b7cbd6e0a1d632257a0267a0053e474239de71 on master.

coveralls commented 7 years ago

Coverage increased (+0.2%) to 83.549% when pulling 7f855d025a0e45581e241fbc462c251c8da22a57 on fix_multiple_stfs into d3b7cbd6e0a1d632257a0267a0053e474239de71 on master.

coveralls commented 7 years ago

Coverage increased (+0.2%) to 83.549% when pulling 7f855d025a0e45581e241fbc462c251c8da22a57 on fix_multiple_stfs into d3b7cbd6e0a1d632257a0267a0053e474239de71 on master.

coveralls commented 7 years ago

Coverage increased (+0.2%) to 83.549% when pulling 7f855d025a0e45581e241fbc462c251c8da22a57 on fix_multiple_stfs into d3b7cbd6e0a1d632257a0267a0053e474239de71 on master.