Skip to content

Remove nwis.py functions with defunct services#216

Open
ehinman wants to merge 5 commits intoDOI-USGS:mainfrom
ehinman:remove-gwlevels-measurements
Open

Remove nwis.py functions with defunct services#216
ehinman wants to merge 5 commits intoDOI-USGS:mainfrom
ehinman:remove-gwlevels-measurements

Conversation

@ehinman
Copy link
Collaborator

@ehinman ehinman commented Feb 25, 2026

Add name errors and deprecation language for:
get_gwlevels
get_discharge_measurements
get_pmcodes
get_water_use

I did not edit the example notebooks from Hydroshare. Are these still being used? Should they be updated or removed?

@ehinman ehinman requested a review from thodson-usgs February 25, 2026 16:25
@thodson-usgs thodson-usgs changed the title Remove nwis.py functions with deprecated services Remove nwis.py functions with defunct services Feb 25, 2026
Copy link
Collaborator

@thodson-usgs thodson-usgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't confirmed whether these APIs are defunct. Assuming they are, I approve.

Please note that deprecated does not mean defunct, and it's important to keep them straight so we don't confuse users.

Comment on lines 152 to 154
Get discharge measurements from the waterdata service -
deprecated, use `get_field_measurements()` in the waterdata
module.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Get discharge measurements from the waterdata service -
deprecated, use `get_field_measurements()` in the waterdata
module.
This endpoint is defunct, use `get_field_measurements()` in the waterdata
module.

Rephrase as you like.

@ehinman ehinman marked this pull request as ready for review February 26, 2026 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants