-
Notifications
You must be signed in to change notification settings - Fork 23
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
Implementation of median amplitude normalization. #261
Conversation
Implementation of wavefield normalization from PR #252. Adding new feature (median-based normalization), and fixing station_amplitude.
Improved degree's distance so that it uses latitude/longitude instead of m_to_deg.
Fixed typo.
Very strange, here is the main problem (some kind of scipy regression?):
For the full traceback: |
There is also a nonfatal typo, from a while back: mtuq/data/examples/unpack.bash Line 12 in 5ebdd79
|
Scipy 1.13.0 was released on April 2nd, so not impossible that the error is coming from there. I'll work on it. I'll also include the unpack fix. |
Removed extra character in one of the file's name. Would prevent unpacking otherwise.
The problem seems to be coming from the dictionary entry in the obspy plugin system. I've tested it by modifying the
and the taper function called in The new doc for |
Scipy version fix to avoid conflict with obspy. (scipy.signal.hann is deprecated since scipy 1.13.0 and called in obspy).
Very helpful thanks! |
See PR #252 for complete discussion.
This also comes with minor fixes for the waveform plots.