-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add converting batches to NetCDF and saving them out #238
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #238 +/- ##
==========================================
+ Coverage 78.60% 79.05% +0.45%
==========================================
Files 129 130 +1
Lines 5693 5868 +175
==========================================
+ Hits 4475 4639 +164
- Misses 1218 1229 +11 ☔ View full report in Codecov by Sentry. |
This has the start of windnet forecasting, but mostly is just adding how to combine and uncombine the output from PVNet datapipe to save to NetCDF, and load it from that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
549be2f
to
21765b7
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
21765b7
to
f0d43f3
Compare
Pull Request
Description
This PR adds support for converting multiple input dataarrays/datasets into a single Dataset to save it out as NetCDF. This is to help with #231 and make batches more interpretable on disk .
Fixes #237
How Has This Been Tested?
Unit tests
Checklist: