openclimatefix / ocf_datapipes

OCF's DataPipe based dataloader for training and inference
MIT License
13 stars 11 forks source link

fix for unittest #211

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Update test to make sure to check_last_timestamp when loading satellite data

How Has This Been Tested?

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #211 (dd97291) into main (69b86c5) will decrease coverage by 0.06%. The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
- Coverage   81.39%   81.34%   -0.06%     
==========================================
  Files         128      128              
  Lines        5548     5549       +1     
==========================================
- Hits         4516     4514       -2     
- Misses       1032     1035       +3     
Impacted Files Coverage Δ
ocf_datapipes/load/satellite.py 90.90% <100.00%> (+0.09%) :arrow_up:

... and 1 file with indirect coverage changes

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