From b824ff5d903ea765b1b12f9e1257ea0870ae9fdd Mon Sep 17 00:00:00 2001
From: braddf <bradbdf@gmail.com>
Date: Wed, 18 Dec 2024 15:57:01 +0000
Subject: [PATCH] update comments re N-hour chart aggregations

---
 .../components/charts/gsp-pv-remix-chart/index.tsx    |  4 ----
 .../charts/gsp-pv-remix-chart/use-get-gsp-data.ts     | 11 +++++++----
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/index.tsx b/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/index.tsx
index 14cf847d..2e75e858 100644
--- a/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/index.tsx
+++ b/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/index.tsx
@@ -57,10 +57,6 @@ const GspPvRemixChart: FC<{
     gspForecastDataOneGSP,
     gspNHourData
   } = useGetGspData(gspId);
-  // TODO – temp reset; if aggregation is zones, make sure data is all set
-  if ([NationalAggregation.DNO, NationalAggregation.zone].includes(nationalAggregationLevel)) {
-    gspNHourData = [];
-  }
   // const gspData = fcAll?.forecasts.find((fc) => fc.location.gspId === gspId);
   const gspInstalledCapacity = gspLocationInfo?.[0]?.installedCapacityMw;
   const gspName = gspLocationInfo?.[0]?.regionName;
diff --git a/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/use-get-gsp-data.ts b/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/use-get-gsp-data.ts
index 5cedb3a9..ce0823fc 100644
--- a/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/use-get-gsp-data.ts
+++ b/apps/nowcasting-app/components/charts/gsp-pv-remix-chart/use-get-gsp-data.ts
@@ -128,13 +128,16 @@ const useGetGspData = (gspId: number | string) => {
     ];
   }
 
-  // TODO: nHour with aggregation
+  // TODO: nHour with aggregation when /forecast/all API endpoint has new forecast_horizon_minutes param
   const nMinuteForecast = nHourForecast * 60;
-  const { data: gspNHourData, error: pvNHourError } = useLoadDataFromApi<ForecastData>(
-    show4hView
+  const { data: gspNHourDataRaw, error: pvNHourError } = useLoadDataFromApi<
+    components["schemas"]["ForecastValue"][]
+  >(
+    show4hView && !isZoneAggregation
       ? `${API_PREFIX}/solar/GB/gsp/${gspId}/forecast?forecast_horizon_minutes=${nMinuteForecast}&historic=true&only_forecast_values=true`
       : null
   );
+  let gspNHourData = gspNHourDataRaw || [];
 
   return {
     errors: [
@@ -144,7 +147,7 @@ const useGetGspData = (gspId: number | string) => {
       gspLocationError,
       pvNHourError
     ].filter((e) => !!e),
-    gspNHourData: gspNHourData,
+    gspNHourData,
     pvRealDataIn,
     pvRealDataAfter,
     gspForecastDataOneGSP,