Closed tclements closed 4 years ago
Wow, thank you! This is great!
I need a git push --force
to update dev
before I can merge your PR. Hopefully nothing catastrophic happens...
It's no big deal if IRISPH5 doesn't immediately support SAC and SEGY, by the way.
Ugh, looks like that created a conflict with the resample!
rewrite. One moment...
Great work, @tclements. All tests passed and CodeCov is virtually unchanged. I'll PR dev
into master
tonight, and we can work on SAC and SEGY support later.
This pull request adds IRISPH5 dataselect capabilities as addressed in #52. I can confirm that mseed and geocsv requests work. sac and segy requests will give fail at the moment (tests included). This PR does not cover IRISPH5 shot or receiver requests.