openclimatefix / ocf_datapipes

OCF's DataPipe based dataloader for training and inference
MIT License
13 stars 11 forks source link

Add WindNet Datapipe #249

Closed jacobbieker closed 7 months ago

jacobbieker commented 7 months ago

Pull Request

Description

Fixes WindNet to properly work with sensor data, and removes some assumptions of coordinate system in the data pipeline.

Part of this partly updates #12

How Has This Been Tested?

Unit tests

Checklist:

codecov[bot] commented 7 months ago

Codecov Report

Attention: 40 lines in your changes are missing coverage. Please review.

Comparison is base (6a1adf2) 78.70% compared to head (ea18cc5) 78.74%. Report is 1 commits behind head on main.

:exclamation: Current head ea18cc5 differs from pull request most recent head f674132. Consider uploading reports for the commit f674132 to get more accurate results

Files Patch % Lines
ocf_datapipes/training/common.py 57.14% 18 Missing :warning:
ocf_datapipes/training/windnet.py 58.82% 14 Missing :warning:
ocf_datapipes/convert/numpy/batch/sensor.py 62.50% 6 Missing :warning:
ocf_datapipes/select/select_loc_and_t0.py 75.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #249 +/- ## ========================================== + Coverage 78.70% 78.74% +0.04% ========================================== Files 131 132 +1 Lines 5880 5929 +49 ========================================== + Hits 4628 4669 +41 - Misses 1252 1260 +8 ```

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