eqcorrscan / EQcorrscan

Earthquake detection and analysis in Python.
https://eqcorrscan.readthedocs.io/en/latest/
Other
167 stars 86 forks source link

Drop last sample, not first #581

Closed calum-chamberlain closed 4 months ago

calum-chamberlain commented 4 months ago

What does this PR do?

As identified in #573 - the first sample is sometimes dropped and the starttime is not adjusted leading to one-sample-out errors. I don't know why that was left in and the tests pass if we drop the last sample instead.

Why was it initiated? Any relevant Issues?

573

PR Checklist