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

Waveform plot upgrade #278

Merged
merged 6 commits into from
Nov 14, 2024
Merged

Waveform plot upgrade #278

merged 6 commits into from
Nov 14, 2024

Conversation

thurinj
Copy link
Member

@thurinj thurinj commented Nov 12, 2024

I finalized the implementation of a waveform plotting option when love waves are decoupled from rayleigh waves (different misfit function and data processing).

It comes with:

  • plot_waveforms3 and plot_data_greens3 functions, which are natural extensions of plot_waveforms[1,2] and plot_data_greens[1,2].
  • Modified header parsing to change behavior depending on the extra arguments.
  • Inclusion of new methods in the documentation

This update does not change the behavior of plot_waveforms[1,2]andplot_data_greens[1,2]`.

plot_data_greens3 expand on the behavior of  plot_data_greens1 and plot_data_greens2, and the accompanying functions plot_waveforms3 naturally complements it.

I've also added a pruning method for the synthetics, which fix the issue of not having all components in the misfit.
The extra fields for the header preparation are know passed as keyword arguments.
…than an integer counter.

This also fixes the numbering that was not displayed properly on the plot_data_greens3 function.
Fix documentation in `plot_waveforms3` and `plot_data_greens3` functions.
@thurinj thurinj requested a review from rmodrak November 12, 2024 01:06
@thurinj thurinj merged commit 943f474 into master Nov 14, 2024
1 check passed
@thurinj thurinj deleted the Waveform_plot_upgrade branch November 14, 2024 22:52
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.

1 participant