openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

Issue/fix migration #175

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Fix for when upgrading the model, id gets pushed

How Has This Been Tested?

Ci test + new test

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #175 (6c34357) into main (7cb7d2c) will increase coverage by 0.00%. The diff coverage is 100.00%.

:exclamation: Current head 6c34357 differs from pull request most recent head 04531db. Consider uploading reports for the commit 04531db to get more accurate results

@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   94.97%   94.97%           
=======================================
  Files          25       25           
  Lines        1532     1533    +1     
=======================================
+ Hits         1455     1456    +1     
  Misses         77       77           
Impacted Files Coverage Δ
nowcasting_datamodel/save/update.py 99.01% <100.00%> (+<0.01%) :arrow_up:

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