fix again

Author: logan-han

app.py

@@ -165,12 +165,13 @@ def get_weather_forecast():
     if not weather_client or not weather_analyser:
         return None
 
-    # Check cache (update every 30 minutes)
+    # Check cache (update every 5 minutes - matches frontend refresh)
+    # 288 calls/day max, well under 1000/day API limit
     cache_age = None
     if weather_forecast_cache["last_update"]:
         cache_age = (datetime.now() - weather_forecast_cache["last_update"]).total_seconds()
 
-    if cache_age is None or cache_age > 1800:  # 30 minutes
+    if cache_age is None or cache_age > 300:  # 5 minutes
         try:
             forecast = weather_client.get_forecast()
             # Get panel capacity for solar estimates
@@ -188,6 +189,7 @@ def get_weather_forecast():
                 capacity_kw = api_inverter_kw * 1.25  # Use API inverter capacity with oversizing ratio
             else:
                 capacity_kw = 0
+            logger.info(f"Solar capacity for forecast: {capacity_kw} kW (panel={panel_kw}, inverter_config={inverter_kw}, inverter_api={api_inverter_kw})")
             # Pass weather_client for hourly solar calculations
             forecast = weather_analyser.analyse_forecast(
                 forecast,
@@ -637,6 +639,10 @@ def complete_setup():
 
         save_config()
 
+        # Clear weather cache to apply new capacity settings
+        weather_forecast_cache["forecast"] = None
+        weather_forecast_cache["last_update"] = None
+
         # Reinitialize clients
         init_client()
         init_weather_client()

templates/index.html

@@ -835,7 +835,7 @@ <h2>Schedule Settings</h2>
             <div class="schedule-grid">
                 <!-- Force Discharge Window -->
                 <div class="schedule-section">
-                    <div class="schedule-section-title">⚡ Force Discharge</div>
+                    <div class="schedule-section-title"><span style="color: #4ecca3;">*</span> Force Discharge</div>
                     <form id="scheduleForm" class="schedule-form">
                         <div class="schedule-row">
                             <div class="setting-item">
@@ -865,7 +865,7 @@ <h2>Schedule Settings</h2>
                     <div class="schedule-section-title">
                         <label style="display: flex; align-items: center; gap: 6px; cursor: pointer;">
                             <input type="checkbox" id="freeEnergyEnabled">
-                            🔋 Free Energy Charge
+                            <span style="color: #3498db;">⚡</span> Free Energy Charge
                         </label>
                     </div>
                     <form id="freeEnergySettingsForm" class="schedule-form">

weather_client.py

@@ -32,7 +32,7 @@ def __init__(self, api_key: str, city_name: str):
         self.base_url = "https://api.openweathermap.org/data/2.5"
         self._cache = {}
         self._cache_time = None
-        self._cache_duration = 1800  # 30 minutes cache
+        self._cache_duration = 300  # 5 minutes cache
 
     def _geocode_city(self) -> bool:
         """Geocode city_name to get latitude and longitude. Returns True if successful."""
@@ -115,10 +115,12 @@ def estimate_solar_output_hourly(self, panel_capacity_kw: float, date_str: str)
             Estimated kWh for the day, or None if hourly data unavailable
         """
         if "hourly_data" not in self._cache:
+            logger.debug(f"No hourly_data in cache for solar estimate")
             return None
 
         hourly_data = self._cache["hourly_data"].get(date_str, [])
         if not hourly_data:
+            logger.debug(f"No hourly data for date {date_str}, available dates: {list(self._cache['hourly_data'].keys())}")
             return None
 
         # Base daily production varies by latitude (peak sun hours equivalent)
@@ -715,6 +717,6 @@ def should_skip_discharge(
                     else:
                         return False, f"Good solar forecast ({estimated_solar:.1f} kWh on {tomorrow.get('day_name', 'Tomorrow')})"
                 else:
-                    return False, "Solar prediction not available (configure panel capacity)"
+                    return False, "Solar prediction not available (no hourly forecast data)"
 
         return False, "Solar threshold not configured"