openclimatefix / uk-pv-forecast-blend

Service to blend forecast together
0 stars 0 forks source link

blend plevels like we do for main variable #13

Closed peterdudfield closed 9 months ago

peterdudfield commented 10 months ago

Pull Request

Description

Blend plevel like we do main expectation value (or the mean value). This means 0-7 hours are pvenet 7-8 are mix 8 onwards are National_xg

Before we were just taking National_xg probability levels

before (from pro)

Screenshot 2023-10-04 at 18 22 30

after (from dev- after running locally)

Screenshot 2023-10-04 at 18 22 20

Fixes #12

How Has This Been Tested?

Checklist:

codecov[bot] commented 10 months ago

Codecov Report

Merging #13 (e1ca8b6) into main (9a00f58) will increase coverage by 1.22%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   95.35%   96.58%   +1.22%     
==========================================
  Files           4        4              
  Lines         323      322       -1     
==========================================
+ Hits          308      311       +3     
+ Misses         15       11       -4     
Files Coverage Δ
forecast_blend/app.py 92.75% <ø> (ø)
forecast_blend/blend.py 98.18% <100.00%> (+3.53%) :arrow_up:
forecast_blend/utils.py 97.24% <100.00%> (ø)

... and 1 file with indirect coverage changes

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