openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

Issue/adjust #151

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

add adjust method to save, this

Fixes #

How Has This Been Tested?

addeds tests + normal CI tests

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #151 (5c115e5) into main (a425247) will decrease coverage by 0.99%. The diff coverage is 96.77%.

:exclamation: Current head 5c115e5 differs from pull request most recent head 1bd2c70. Consider uploading reports for the commit 1bd2c70 to get more accurate results

@@            Coverage Diff             @@
##             main     #151      +/-   ##
==========================================
- Coverage   95.86%   94.87%   -0.99%     
==========================================
  Files          21       22       +1     
  Lines        1210     1289      +79     
==========================================
+ Hits         1160     1223      +63     
- Misses         50       66      +16     
Impacted Files Coverage Δ
nowcasting_datamodel/national.py 90.90% <92.30%> (-1.20%) :arrow_down:
nowcasting_datamodel/models/forecast.py 83.92% <95.00%> (+1.37%) :arrow_up:
nowcasting_datamodel/save/adjust.py 97.50% <97.50%> (ø)
nowcasting_datamodel/__init__.py 100.00% <100.00%> (ø)
nowcasting_datamodel/fake.py 100.00% <100.00%> (ø)
nowcasting_datamodel/models/metric.py 97.01% <100.00%> (+0.04%) :arrow_up:
nowcasting_datamodel/models/utils.py 100.00% <100.00%> (ø)
nowcasting_datamodel/read/read_metric.py 100.00% <100.00%> (ø)
nowcasting_datamodel/save/save.py 100.00% <100.00%> (ø)
nowcasting_datamodel/save/update.py 98.96% <100.00%> (ø)
... and 2 more

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