Skip to content

add nwp source to api - #311

Open
peterdudfield wants to merge 3 commits into
mainfrom
add-api-nwp-provider
Open

add nwp source to api#311
peterdudfield wants to merge 3 commits into
mainfrom
add-api-nwp-provider

Conversation

@peterdudfield

Copy link
Copy Markdown
Contributor

Pull Request

Description

Added nwp source to api

This helps with openclimatefix/global-solar-forecast#7

How Has This Been Tested?

  • CI tests

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

@sentry

sentry Bot commented Sep 11, 2025

Copy link
Copy Markdown

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: api/v1/api.py

Function Unhandled Issue
forecast KeyError: 'timestamp' /forecast/
Event Count: 10
forecast TypeError: '<' not supported between instances of 'int' and 'datetime.datetime' ...
Event Count: 8
forecast UnboundLocalError: cannot access local variable 'live_generation_df' where it is not associated with a value ...
Event Count: 2
forecast NameError: name 'data' is not defined /forecast/
Event Count: 1
forecast AttributeError: 'NoneType' object has no attribute 'values' ...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant