openclimatefix / nowcasting_forecast

Making live forecasts for the nowcasting project
MIT License
5 stars 2 forks source link

Issue/fix tests #187

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

How Has This Been Tested?

CI tests

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #187 (2db30bf) into main (4a97992) will increase coverage by 0.02%. The diff coverage is 100.00%.

:exclamation: Current head 2db30bf differs from pull request most recent head 6bba57c. Consider uploading reports for the commit 6bba57c to get more accurate results

@@            Coverage Diff             @@
##             main     #187      +/-   ##
==========================================
+ Coverage   90.66%   90.68%   +0.02%     
==========================================
  Files          15       15              
  Lines         739      741       +2     
==========================================
+ Hits          670      672       +2     
  Misses         69       69              
Impacted Files Coverage Δ
nowcasting_forecast/models/cnn/model.py 98.01% <ø> (ø)
nowcasting_forecast/models/nwp_solar_simple.py 96.66% <ø> (ø)
nowcasting_forecast/models/sun.py 100.00% <ø> (ø)
nowcasting_forecast/__init__.py 100.00% <100.00%> (ø)
nowcasting_forecast/app.py 86.95% <100.00%> (ø)
nowcasting_forecast/models/cnn/cnn.py 97.14% <100.00%> (ø)
nowcasting_forecast/models/utils.py 92.06% <100.00%> (+0.12%) :arrow_up:

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

peterdudfield commented 1 year ago

this was for. - https://github.com/openclimatefix/nowcasting_forecast/issues/186