Closed jacobbieker closed 10 months ago
Merging #229 (39d7bbb) into main (2c39650) will increase coverage by
0.02%
. Report is 2 commits behind head on main. The diff coverage is77.77%
.:exclamation: Current head 39d7bbb differs from pull request most recent head d17bdc7. Consider uploading reports for the commit d17bdc7 to get more accurate results
@@ Coverage Diff @@
## main #229 +/- ##
==========================================
+ Coverage 80.11% 80.13% +0.02%
==========================================
Files 127 127
Lines 5611 5614 +3
==========================================
+ Hits 4495 4499 +4
+ Misses 1116 1115 -1
Files | Coverage Δ | |
---|---|---|
ocf_datapipes/training/common.py | 98.08% <ø> (ø) |
|
ocf_datapipes/training/example/nwp_pv.py | 89.28% <100.00%> (ø) |
|
ocf_datapipes/load/nwp/nwp.py | 92.85% <71.42%> (-4.58%) |
:arrow_down: |
... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Pull Request
Description
This adds support for loading different providers of NWP data in the common datapipe utility, focused primarily on PVNet forecasting.
Fixes #
How Has This Been Tested?
Unit tests
Checklist: