owm issue fix

Author: logan-han

app.py

@@ -199,7 +199,7 @@ def get_weather_forecast():
             )
             weather_forecast_cache["forecast"] = forecast
             weather_forecast_cache["last_update"] = datetime.now()
-            logger.info(f"Weather forecast updated: {forecast.get('consecutive_bad_days', 0)} consecutive bad days")
+            logger.info(f"Weather forecast updated successfully")
         except Exception as e:
             logger.error(f"Failed to fetch weather forecast: {e}")
             # Return cached data if available

weather_client.py

@@ -333,8 +333,9 @@ def _make_request_with_retry(self, url: str, params: dict, max_retries: int = 2)
 
     def get_forecast(self) -> Dict[str, Any]:
         """
-        Get 8-day weather forecast (today + 7 days)
-        Uses OpenWeatherMap One Call API 3.0 for daily forecasts
+        Get 5-day weather forecast with 3-hourly data for solar predictions.
+        Uses the free tier 5-day/3-hour forecast API which provides
+        the hourly data needed for accurate solar output estimates.
         """
         if self._is_cache_valid() and "forecast" in self._cache:
             logger.debug("Using cached forecast data")
@@ -347,41 +348,8 @@ def get_forecast(self) -> Dict[str, Any]:
                 "error": f"Could not geocode city: {self.city_name}"
             }
 
-        try:
-            # Use One Call API 3.0 for daily forecast
-            url = "https://api.openweathermap.org/data/3.0/onecall"
-            params = {
-                "lat": self.latitude,
-                "lon": self.longitude,
-                "appid": self.api_key,
-                "units": "metric",
-                "exclude": "minutely,hourly,alerts"
-            }
-
-            location_str = self.city_name if self.city_name else f"({self.latitude}, {self.longitude})"
-            logger.info(f"Fetching weather forecast for {location_str}")
-            response = self._make_request_with_retry(url, params)
-
-            # If One Call 3.0 fails (requires subscription), fallback to 2.5
-            if response.status_code == 401 or response.status_code == 403:
-                logger.warning("One Call API 3.0 not available, trying 2.5")
-                return self._get_forecast_legacy()
-
-            response.raise_for_status()
-            data = response.json()
-
-            forecast = self._parse_onecall_forecast(data)
-            self._cache["forecast"] = forecast
-            self._cache_time = datetime.now()
-
-            return forecast
-
-        except WeatherAPIError as e:
-            logger.warning(f"One Call API failed: {e}, trying legacy API")
-            return self._get_forecast_legacy()
-        except requests.exceptions.HTTPError as e:
-            logger.warning(f"One Call API HTTP error: {e}, trying legacy API")
-            return self._get_forecast_legacy()
+        # Use legacy 5-day/3-hour API - free tier and provides hourly data for solar calc
+        return self._get_forecast_legacy()
 
     def _get_forecast_legacy(self) -> Dict[str, Any]:
         """