Skip to content
This repository was archived by the owner on Sep 11, 2023. It is now read-only.

Conversation

@peterdudfield
Copy link
Contributor

Pull Request

Description

fix for forecasting error

Helps with openclimatefix-archives/nowcasting_forecast#207

How Has This Been Tested?

CI tests, but not sure it will solve the bug

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@peterdudfield peterdudfield self-assigned this Aug 23, 2023
@codecov
Copy link

codecov bot commented Aug 23, 2023

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% <ø> (ø)

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

@peterdudfield peterdudfield merged commit 9e0fbc3 into main Aug 23, 2023
@peterdudfield peterdudfield deleted the issue/bug-int branch August 23, 2023 07:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants