Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a national end point, that provides elexon BMRS solar foreacsts from elexonpy package #347

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
e125c2a
added import statements of elexonpy
14Richa Jul 17, 2024
1ab3837
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jul 17, 2024
aec73d2
resolve conflicts
14Richa Jul 18, 2024
9b302f9
Merge branch '14Richa/solar-forecast-endpoint' of github.com:14Richa/…
14Richa Jul 18, 2024
bae31a9
added API for solar forecast
14Richa Jul 18, 2024
9158800
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jul 18, 2024
81af713
minor fix
14Richa Jul 19, 2024
9ca6cf0
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jul 19, 2024
746d979
minor fix
14Richa Jul 19, 2024
f6cad5d
Merge branch '14Richa/solar-forecast-endpoint' of github.com:14Richa/…
14Richa Jul 19, 2024
8b988ab
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jul 19, 2024
299497d
changed function name
14Richa Jul 19, 2024
f03e377
Merge branch '14Richa/solar-forecast-endpoint' of github.com:14Richa/…
14Richa Jul 19, 2024
0193ad0
added filter functionality
14Richa Jul 19, 2024
e4b19e4
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jul 19, 2024
bc5cece
added API link
14Richa Jul 19, 2024
a1ddc32
Merge branch '14Richa/solar-forecast-endpoint' of github.com:14Richa/…
14Richa Jul 19, 2024
f86c36b
minor fix
14Richa Jul 20, 2024
25a345b
remove try except block
14Richa Jul 20, 2024
f254216
changed bmrs to elexon
14Richa Jul 20, 2024
67a2293
minor fix
14Richa Jul 23, 2024
e01ea24
resolving pre hook
14Richa Jul 23, 2024
b12e8c2
minor fix in docstring
14Richa Jul 24, 2024
c40b4ec
removed router
14Richa Jul 24, 2024
7d21b13
minor fix in naming
14Richa Jul 24, 2024
841554a
added elexonpy in requirement.txt file
14Richa Jul 24, 2024
448bd81
resolve hook error
14Richa Jul 25, 2024
8e8167a
minor fix
14Richa Jul 25, 2024
e734329
added response model
14Richa Jul 27, 2024
0d0cc1c
fixed unit test
14Richa Jul 27, 2024
e53d7a8
remove plevel
14Richa Jul 27, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,4 @@ slowapi
pathy==0.10.3
fsspec
s3fs
elexonpy
1 change: 0 additions & 1 deletion src/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,6 @@ async def add_process_time_header(request: Request, call_next):
# Dependency
v0_route_solar = "/v0/solar/GB"
v0_route_system = "/v0/system/GB"

app.include_router(national_router, prefix=f"{v0_route_solar}/national")
app.include_router(gsp_router, prefix=f"{v0_route_solar}/gsp")
app.include_router(status_router, prefix=f"{v0_route_solar}")
Expand Down
123 changes: 93 additions & 30 deletions src/national.py
Original file line number Diff line number Diff line change
@@ -1,10 +1,14 @@
"""National API routes"""

import os
from datetime import datetime, timedelta
from typing import List, Optional, Union

import pandas as pd
import structlog
from fastapi import APIRouter, Depends, HTTPException, Request, Security
from elexonpy.api.generation_forecast_api import GenerationForecastApi
from elexonpy.api_client import ApiClient
from fastapi import APIRouter, Depends, HTTPException, Query, Request, Security
from fastapi_auth0 import Auth0User
from nowcasting_datamodel.read.read import get_latest_forecast_for_gsps
from sqlalchemy.orm.session import Session
Expand All @@ -16,19 +20,98 @@
get_session,
get_truth_values_for_a_specific_gsp_from_database,
)
from pydantic_models import NationalForecast, NationalForecastValue, NationalYield
from pydantic_models import (
NationalForecast,
NationalForecastValue,
NationalYield,
SolarForecastResponse,
SolarForecastValue,
)
from utils import N_CALLS_PER_HOUR, filter_forecast_values, format_datetime, format_plevels, limiter

logger = structlog.stdlib.get_logger()


adjust_limit = float(os.getenv("ADJUST_MW_LIMIT", 0.0))
get_plevels = bool(os.getenv("GET_PLEVELS", True))

router = APIRouter(
tags=["National"],
)

# Initialize Elexon API client
api_client = ApiClient()
forecast_api = GenerationForecastApi(api_client)


peterdudfield marked this conversation as resolved.
Show resolved Hide resolved
@router.get("/elexon", summary="Get elexon Solar Forecast")
@limiter.limit(f"{N_CALLS_PER_HOUR}/hour")
def get_elexon_forecast(
request: Request,
start_datetime_utc: datetime = Query(
default=datetime.utcnow() - timedelta(days=3), description="Start date and time in UTC"
),
end_datetime_utc: datetime = Query(
default=datetime.utcnow() + timedelta(days=3), description="End date and time in UTC"
),
process_type: str = Query("Day Ahead", description="Process type"),
):
"""
Fetch elexon Solar and wind(?) forecasts from the Elexon API.

Args:
request (Request): The request object containing metadata about the HTTP request.
start_datetime_utc (datetime): The start date and time in UTC.
end_datetime_utc (datetime): The end date and time in UTC.
process_type (str): The type of process (e.g., 'Day Ahead').

Returns:
SolarForecastResponse: The forecast data wrapped in a SolarForecastResponse model.
"""
try:
14Richa marked this conversation as resolved.
Show resolved Hide resolved
response = forecast_api.forecast_generation_wind_and_solar_day_ahead_get(
_from=start_datetime_utc.isoformat(),
to=end_datetime_utc.isoformat(),
process_type=process_type,
format="json",
)

if not response.data:
return SolarForecastResponse(data=[])

df = pd.DataFrame([item.to_dict() for item in response.data])
logger.debug("DataFrame Columns: %s", df.columns)
logger.debug("DataFrame Sample: %s", df.head())

# Filter to include only solar forecasts
solar_df = df[df["business_type"] == "Solar generation"]
logger.debug("Filtered Solar DataFrame: %s", solar_df.head())

forecast_values = []
for _, row in solar_df.iterrows():
try:
forecast_values.append(
SolarForecastValue(
timestamp=pd.to_datetime(row["publish_time"]).to_pydatetime(),
expected_power_generation_megawatts=row.get("quantity"),
plevels=None,
)
)
except KeyError as e:
logger.error("KeyError: %s. Data: %s", str(e), row)
raise HTTPException(status_code=500, detail="Internal Server Error")
except Exception as e:
logger.error(
"Error during DataFrame to Model conversion: %s. Data: %s", str(e), row
)
raise HTTPException(status_code=500, detail="Internal Server Error")

result = SolarForecastResponse(data=forecast_values)
return result

except Exception as e:
logger.error("Unhandled exception: %s", str(e))
raise HTTPException(status_code=500, detail="Internal Server Error")

14Richa marked this conversation as resolved.
Show resolved Hide resolved

@router.get(
"/forecast",
Expand All @@ -47,7 +130,9 @@ def get_national_forecast(
end_datetime_utc: Optional[str] = None,
creation_limit_utc: Optional[str] = None,
) -> Union[NationalForecast, List[NationalForecastValue]]:
"""Get the National Forecast
"""

Fetch national forecasts.

This route returns the most recent forecast for each _target_time_.

Expand All @@ -67,6 +152,9 @@ def get_national_forecast(
- **creation_utc_limit**: optional, only return forecasts made before this datetime.
Note you can only go 7 days back at the moment

Returns:
dict: The national forecast data.

"""
logger.debug("Get national forecasts")

Expand Down Expand Up @@ -182,35 +270,10 @@ def get_national_pvlive(

#### Parameters
- **regime**: can choose __in-day__ or __day-after__
"""

"""
logger.info(f"Get national PV Live estimates values " f"for regime {regime} for {user}")

return get_truth_values_for_a_specific_gsp_from_database(
session=session, gsp_id=0, regime=regime
)


@router.get(
"/bmrs",
response_model=dict,
# dependencies=[Depends(get_auth_implicit_scheme())],
summary="Get BMRS Forecast",
)
# @cache_response
@limiter.limit(f"{N_CALLS_PER_HOUR}/hour")
def get_bmrs_forecast(
request: Request,
# session: Session = Depends(get_session),
# user: Auth0User = Security(get_user()),
) -> dict:
"""

This route returns the most recent BMRS forecast for each _target_time_.

#### Parameters

"""
logger.debug("Get bmrs forecast")

return {"message": "This route is not yet implemented. Please check back later."}
24 changes: 23 additions & 1 deletion src/pydantic_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

from nowcasting_datamodel.models import Forecast, ForecastSQL, ForecastValue, Location, LocationSQL
from nowcasting_datamodel.models.utils import EnhancedBaseModel
from pydantic import Field, validator
from pydantic import BaseModel, Field, validator

logger = logging.getLogger(__name__)

Expand Down Expand Up @@ -214,3 +214,25 @@ class NationalForecast(Forecast):
"""One Forecast of generation at one timestamp"""

forecast_values: List[NationalForecastValue] = Field(..., description="List of forecast values")


class SolarForecastValue(BaseModel):
"""Represents a single solar forecast entry"""

timestamp: datetime = Field(..., description="Timestamp of the forecast")
expected_power_generation_megawatts: Optional[float] = Field(
None, ge=0, description="Expected power generation in megawatts"
)

@validator("expected_power_generation_megawatts")
def result_check(cls, v):
"""Round to 2 decimal places"""
if v is not None:
return round(v, 2)
return v


class SolarForecastResponse(BaseModel):
"""Wrapper for a list of solar forecast values"""

data: List[SolarForecastValue] = Field(..., description="List of solar forecast values")
63 changes: 63 additions & 0 deletions src/tests/test_elexon_forecast.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,63 @@
import pytest
import requests
import requests_mock

API_URL = "/v0/solar/GB/national/elexon"


@pytest.fixture
def mock_data():
return {
"data": [
{
"timestamp": "2024-07-24T16:45:09+00:00",
"expected_power_generation_megawatts": 0,
"plevels": None,
},
{
"timestamp": "2024-07-24T16:45:09+00:00",
"expected_power_generation_megawatts": 0,
"plevels": None,
},
]
}


def test_get_elexon_forecast_with_data(mock_data):
with requests_mock.Mocker() as m:
url = (
f"{API_URL}?start_datetime_utc=2024-07-22T10:56:59.194610"
f"&end_datetime_utc=2024-07-28T10:56:59.194680"
f"&process_type=Day Ahead"
)
m.get(url, json=mock_data, headers={"Content-Type": "application/json"})

response = requests.get(url)
print("Response Headers:", response.headers)
# Assertions
assert response.status_code == 200
assert response.headers.get("Content-Type") == "application/json"
assert response.json() == mock_data


@pytest.fixture
def empty_mock_data():
return {"data": []}


def test_get_elexon_forecast_no_data(empty_mock_data):
with requests_mock.Mocker() as m:
url = (
f"{API_URL}?start_datetime_utc=2024-07-22T10:56:59.194610"
f"&end_datetime_utc=2024-07-28T10:56:59.194680"
f"&process_type=Day Ahead"
)

m.get(url, json=empty_mock_data, headers={"Content-Type": "application/json"})
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to set the actual api up and call the route. See other national tests


response = requests.get(url)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead use response = api_client.get("/v0/solar/GB/national/elexon"), this should use the mocked api you have done above

print("Response Headers:", response.headers)
# Assertions
assert response.status_code == 200
assert response.headers.get("Content-Type") == "application/json"
assert response.json() == empty_mock_data