openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

Issue/fix blend #166

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Fix for blending forecast, needed to reset pandas index as they were not unique Fixes #

How Has This Been Tested?

-TDD, added failing test, then fix it

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #166 (a902014) into main (8ae1122) will increase coverage by 0.01%. The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##             main     #166      +/-   ##
==========================================
+ Coverage   94.84%   94.85%   +0.01%     
==========================================
  Files          23       23              
  Lines        1455     1459       +4     
==========================================
+ Hits         1380     1384       +4     
  Misses         75       75              
Impacted Files Coverage Δ
nowcasting_datamodel/__init__.py 100.00% <100.00%> (ø)
nowcasting_datamodel/read/blend.py 91.85% <100.00%> (+0.24%) :arrow_up:

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