Repository navigation
fix(waterdata): accept field-measurements sublocation_identifier - #436
Merged
thodson-usgs merged 1 commit intoOct 7, 2026
Conversation
thodson-usgs
force-pushed
the
fix/field-measurements-sublocation-identifier
branch
from
October 3, 2026 12:55
f2806d7 to
75bcad3
Compare
thodson-usgs
commented
Oct 3, 2026
thodson-usgs
force-pushed
the
fix/field-measurements-sublocation-identifier
branch
from
October 3, 2026 13:03
75bcad3 to
1f7fc21
Compare
…queryable USGS added sublocation_identifier to the field-measurements and field-measurements-metadata collections at the end of September 2026, which failed the live queryables snapshot monitor. Regenerate the snapshot, and add sublocation_identifier as a named, documented parameter on get_field_measurements and get_field_measurements_metadata.
thodson-usgs
force-pushed
the
fix/field-measurements-sublocation-identifier
branch
from
October 3, 2026 13:06
1f7fc21 to
4e86db6
Compare
thodson-usgs
marked this pull request as ready for review
October 3, 2026 13:07
ehinman
approved these changes
Oct 5, 2026
ehinman
left a comment
ehinman
left a comment
Collaborator
There was a problem hiding this comment.
LGTM. I'm struggling to find a monitoring location in the field measurements/field metadata endpoints that has a non-NA sublocation identifier populated, but I passed sublocation_identifier=None to get_field_measurements() and it returned all rows. Thanks for adding this.
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
2 participants
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
The scheduled live-API run 37105829876 failed in
test_queryables_match_snapshotforfield-measurementsandfield-measurements-metadata: USGS added asublocation_identifierqueryable to both collections. It appeared onfield-measurements-metadataby the 09/30 run and onfield-measurementsby the 10/03 run. No other collection's queryables changed.Following the pattern of #423 (
method_category):tests/data/waterdata_queryables.jsonwith the script inwaterdata_queryables_test.py. The only change is the twosublocation_identifierentries.sublocation_identifieras a named, documented parameter onget_field_measurements()andget_field_measurements_metadata(). Before this, it could only be passed through**queryables.null, which is what the live service returns for those USGS-05427718 records.The live
field-measurements-metadatafailure in the 09/30–10/02 runs was the same cause. Those runs also had thecontinuousfailure that #423 fixed.Testing
pytest tests/(offline): 1242 passed. Coverage is 98.96%, and the ratchet holds.pytest -m live tests/waterdata_queryables_test.py::test_queryables_match_snapshot: 11 passed.sublocation_identifier="UPSTREAM". Each returned onlyUPSTREAMrows.