openclimatefix / nowcasting_dataset

Prepare batches of data for training machine learning solar electricity nowcasting data
https://nowcasting-dataset.readthedocs.io/en/stable/
MIT License
25 stars 6 forks source link

make sure slice on osgb is ints #728

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

fix for forecasting error

Helps with https://github.com/openclimatefix/nowcasting_forecast/issues/207

How Has This Been Tested?

CI tests, but not sure it will solve the bug

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #728 (2b0490d) into main (db2af0c) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #728   +/-   ##
=======================================
  Coverage   92.86%   92.86%           
=======================================
  Files          49       49           
  Lines        3488     3488           
=======================================
  Hits         3239     3239           
  Misses        249      249           
Files Changed Coverage Δ
nowcasting_dataset/data_sources/data_source.py 89.63% <ø> (ø)

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