pySTEPS / pysteps

Python framework for short-term ensemble prediction systems.
https://pysteps.github.io/
BSD 3-Clause "New" or "Revised" License
441 stars 160 forks source link

fix: make sure netcdf exporters can handle list of timesteps #369

Closed RubenImhoff closed 1 month ago

RubenImhoff commented 1 month ago

This PR ensures that the netcdf exporters can handle a list of timesteps next to just an integers number of timesteps. One of pysteps' functionalities is that you can give it specific output timesteps for which the nowcasts should be stored, but it was not yet possible to store these output timesteps as such in a netCDF. This PR tries to fix that.

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.52%. Comparing base (1525178) to head (2abd113). Report is 13 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #369 +/- ## ========================================== + Coverage 83.33% 83.52% +0.18% ========================================== Files 161 159 -2 Lines 12370 12575 +205 ========================================== + Hits 10309 10503 +194 - Misses 2061 2072 +11 ``` | [Flag](https://app.codecov.io/gh/pySTEPS/pysteps/pull/369/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pySTEPS) | Coverage Δ | | |---|---|---| | [unit_tests](https://app.codecov.io/gh/pySTEPS/pysteps/pull/369/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pySTEPS) | `83.52% <100.00%> (+0.18%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pySTEPS#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.