openclimatefix / PVNet

PVnet main repo

MIT License

17 stars 3 forks source link

ocf_datapipes==1.2.44 #52

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Upgrade datamodel, fix for 15 minute data

How Has This Been Tested?

Ci tests
ran locally with changes
[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

codecov[bot] commented 1 year ago

Codecov Report

Merging #52 (d0a7cbb) into main (998ba1e) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage   68.23%   68.23%           
=======================================
  Files          22       22           
  Lines        1615     1615           
=======================================
  Hits         1102     1102           
  Misses        513      513

Impacted Files	Coverage Δ
pvnet/__init__.py	`100.00% <100.00%> (ø)`

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

peterdudfield commented 1 year ago

Ive pushed this one thourgh too, @dfulu but any comments are very welcome