openclimatefix / pv-site-production

Production service for PV site level forecasts
MIT License
1 stars 3 forks source link

Lazy load nwp data #39

Closed simlmx closed 1 year ago

simlmx commented 1 year ago

I recommend reviewing commit by commit and the last one is the most relevant.

peterdudfield commented 1 year ago

Do you know why the docker tests now fail?

Otherwise it looks good

codecov-commenter commented 1 year ago

Codecov Report

Merging #39 (26fb2d8) into main (1e674bb) will increase coverage by 1.32%. The diff coverage is 88.88%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
+ Coverage   91.26%   92.59%   +1.32%     
==========================================
  Files           9       10       +1     
  Lines         252      270      +18     
==========================================
+ Hits          230      250      +20     
+ Misses         22       20       -2     
Impacted Files Coverage Δ
pv_site_production/data/nwp_data_sources.py 69.56% <0.00%> (+1.56%) :arrow_up:
pv_site_production/app.py 95.58% <100.00%> (+1.64%) :arrow_up:
pv_site_production/models/psp.py 100.00% <100.00%> (ø)
pv_site_production/utils/profiling.py 100.00% <100.00%> (ø)

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