Skip to content

Greece OHI#107

Open
thiagovmdon wants to merge 2 commits intomainfrom
greece-ohi
Open

Greece OHI#107
thiagovmdon wants to merge 2 commits intomainfrom
greece-ohi

Conversation

@thiagovmdon
Copy link
Copy Markdown
Collaborator

Related to issue #106

Description

This PR adds a Greece fetcher based on the OpenHI / Enhydris public API. It follows the same overall architecture and project conventions as the recent country fetchers, including cached station metadata, example usage, detailed Sphinx documentation, and unit tests.

Summary

  • add GreeceFetcher for the OpenHI / Enhydris public API
  • support stage_daily_mean, stage_instantaneous, discharge_daily_mean, and discharge_instantaneous
  • export the fetcher from rivretrieve.__init__
  • add cached Greece station metadata, detailed Sphinx docs, example usage, and unit tests

Implementation details

  • discover hydrometric stations from the union of ts_only: variable:stage, ts_only: variable:level, and ts_only: variable:discharge
  • normalize provider Stage and Water Level to RivRetrieve stage variables
  • normalize stage units from cm to m
  • convert provider sentinel values such as -999, -9999, and -6999 to missing values
  • rank compatible groups and timeseries deterministically, preferring checked/public series and falling back when aggregated daily series are empty
  • document that cached provider-derived metadata remain subject to the OpenHI CC BY-SA 4.0 licence

@thiagovmdon thiagovmdon requested a review from kratzert as a code owner March 19, 2026 16:37
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.

1 participant