-
Notifications
You must be signed in to change notification settings - Fork 150
Add forecast classes #1115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Add forecast classes #1115
Conversation
* define base class * add init test * Fix #1127 --------- Co-authored-by: Valentin Gebhart <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* Add ImpactForecast * Add unit tests
* Create hazardForecast base class * Update datetime to timedelta for lead_time * Add tests --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* change leadtime to timedelta * adapt impactforecast docstrings and tests * Remove stray comment --------- Co-authored-by: Lukas Riedel <[email protected]>
* Add HazardForecast.select test --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* Add test for hazardForecast.concat * Skip HazardForecast.concat test --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* Add test for ImpactForecast.select * minor test modification --------- Co-authored-by: Evelyn Mühlhofer <[email protected]>
* Add test stub for ImpactForecast.concat * Use correct command for skipping a test * Fix merge issue
* Add data shape check for HazardForecast, ImpactForecast
Adapt Hazard from_hdf5 and write_hdf5 to forecast attributes
* Add extended select impact forecast test * Add extended hazard forecast test --------- Co-authored-by: Chahan Kropf <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* Return impactForecast object in _return_impact * Return impactForecast object in _return_empty * Add full impactcalc test for impactForecast * Raise value error when computing impact with impact forecast without saving impact matrix * Cosmetics: Improve error message, move test to own class * Write nans for eai_exp and aai_agg when forecast is used --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Chahan Kropf <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* add idx boolean selection for member and leadtime * Add test for weird data types --------- Co-authored-by: Lukas Riedel <[email protected]>
* Add ImpactForecast.select --------- Co-authored-by: Lukas Riedel <[email protected]>
* add HazardForecast.select and extent test --------- Co-authored-by: Eliane Kobler <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
85b92ef to
302be63
Compare
Add min/mean/max methods --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Chahan Kropf <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
* implement changes in Impact IO and ImpactForecast IO * Update tests * Fix small issue when reading H5 for ImpactForecast --------- Co-authored-by: Lukas Riedel <[email protected]>
* Add quantile and median for hazard and impact forecast * Review --------- Co-authored-by: luseverin <[email protected]> Co-authored-by: Chahan Kropf <[email protected]> Co-authored-by: Lukas Riedel <[email protected]>
…ython into forecast-class
* Add impact forecast tutorial * Add 'dim' argument to reductions * Update tests --------- Co-authored-by: Lukas Riedel <[email protected]> Co-authored-by: luseverin <[email protected]> Co-authored-by: Evelyn Mühlhofer <[email protected]> Co-authored-by: Chahan Kropf <[email protected]>
…ython into forecast-class
|
@Evelyn-M How was the |
I downloaded it from the meteoswiss-internal data storage, since the API only permits downloads until -24 hrs. |
|
So it had been reprojected? Does MCH store the data differently than OGD? I am just wondering because it might be important to highlight the caveats when importing datasets with varying CRS and coordinate specifications. |
|
I suppose so, yes. One is the MCH grid commonly used internally for national gridded forecast products, and the OGD data has a different grid and a much larger rectangular extent around Switzerland. |
Changes proposed in this PR:
This PR resolves a major portion of #1118. Note that sub-tasks have been marked as completed when merged into this branch.
PR Author Checklist
develop)PR Reviewer Checklist