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

Permutation and naming fix #255

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

ammcpherson
Copy link
Member

This pull request addresses issues found during further testing of using SPECFEM3D_Cartesian-generated Green's Functions for force inversions. 2 changes have been made:

  1. mtuq/io/clients/SPECFEM3D_SAC.py has been changed to reflect the new naming convention, which is in line with MTUQ's internal coordinate system now. This is also a change for clarity - the new naming convention reminds new users to run 3 simulations for an Up force, a South force, and an East force.
  2. mtuq/greens_tensor/SPECFEM3D.py is now also in line with MTUQ's internal coordinate system. This permutation has passed 5 synthetic tests, and one test using the 2009 Anchorage earthquake data. The comparison between a force inversion using the AxiSEM Green's functions (left) and the SPECFEM3D Green's functions (right) is shown below.
Screen Shot 2024-03-12 at 14 15 47

@rmodrak rmodrak merged commit 86168bd into mtuqorg:master Mar 13, 2024
1 check passed
@ammcpherson ammcpherson deleted the force_gf_update branch March 13, 2024 18:25
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.

2 participants