Add station_codes and picked_stations_only config parameters for waveform downloading - #8
Closed
claudiodsf with Copilot wants to merge 3 commits into
Closed
Add station_codes and picked_stations_only config parameters for waveform downloading#8claudiodsf with Copilot wants to merge 3 commits into
claudiodsf with Copilot wants to merge 3 commits into
Conversation
…form downloading Agent-Logs-Url: https://github.com/SeismicSource/seiscat/sessions/b11546ea-ec31-4962-843d-20f660366f22 Co-authored-by: claudiodsf <1390993+claudiodsf@users.noreply.github.com>
Agent-Logs-Url: https://github.com/SeismicSource/seiscat/sessions/b11546ea-ec31-4962-843d-20f660366f22 Co-authored-by: claudiodsf <1390993+claudiodsf@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
claudiodsf
April 21, 2026 12:25
View session
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
claudiodsf
reviewed
Apr 21, 2026
Member
There was a problem hiding this comment.
This file should not be touched. It's used for the GitHub action that generates a release
Agent-Logs-Url: https://github.com/SeismicSource/seiscat/sessions/fcea3539-9c4c-4e3a-8f13-7f2267f2af44 Co-authored-by: claudiodsf <1390993+claudiodsf@users.noreply.github.com>
Member
|
Branch was renamed to 'copilot/add-waveform-station-filters'. Continuing discussion in the replacement PR: #10 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses feature request #7 by adding two new station selection criteria for downloading waveforms:
1.
station_codes— filter by station nameSimilar to the existing
channel_codesparameter, but for station names. Supports multiple codes and wildcard patterns (?for single character,*for any characters).Example config:
2.
picked_stations_only— restrict to stations with P/S picksWhen set to
True, only stations with at least one P or S-wave arrival in the event catalog are downloaded. This requires a QuakeML event details file to be present (generated byseiscat fetch_details). A warning is shown if the file is missing, and the download falls back to other criteria.Example config:
picked_stations_only = TrueBoth criteria can be combined: if both
station_codesandpicked_stations_onlyare set, only stations that satisfy both conditions are downloaded.Changes
seiscat/config/configspec.conf: two new documented parametersseiscat/fetchdata/event_waveforms_utils.py:check_station()— wildcard pattern matching for station namesget_picked_station_codes()— extracts station codes with P/S picks from a QuakeML fileseiscat/fetchdata/mass_downloader.py:_build_station_restriction()helper that builds the combined station restrictionRestrictionsseiscat/fetchdata/sds.py: applies station restrictions when fetching from local SDS archivetests/test_station_selection.py: 11 new unit tests (wildcard matching and pick extraction)CHANGELOG.md: updated with the new features