Update app.py

app.py

@@ -37,37 +37,42 @@ data_source = st.sidebar.selectbox(
 openweather_historical_readings = []  
 openweather_forecast_readings = []  
 stormglass_marine_readings = []  
-solunar_tide_readings = []   
+solunar_tide_readings = [] 
+
+
+# Initialize session state for historical weather data  
+if 'openweather_historical_readings' not in st.session_state:  
+    st.session_state.openweather_historical_readings = []  
 
 # --- API CALLING FUNCTIONS ---  
 def fetch_openweather_data(lat, lon, datetime_input):  
     """Fetches historical weather data from OpenWeather's One Call API."""  
-    # Construct the URL for the API call  
-    url = f"https://api.openweathermap.org/data/3.0/onecall/timemachine?lat={lat}&lon={lon}&dt={int(datetime_input.timestamp())}&appid={OPENWEATHER_API_KEY}&units=metric"  
     try:  
-        response = requests.get(url)  # Make the API call  
-        response.raise_for_status()  # Raise an error for bad responses  
-        data = response.json()  # Parse JSON response
-        # Log the data
-        logging.debug(f"Data: {data}")
-        current_weather_data = data.get('data', [])[0] if data.get('data', []) else {}
-        processed_entry = {
-            'timestamp': datetime_input,
-            'latitude': lat,
-            'longitude': lon,
-            'temperature': current_weather_data.get('temp'),
-            'humidity': current_weather_data.get('humidity'),
-            'wind_speed': current_weather_data.get('wind_speed')
-            } 
-        logging.debug(f"Processed Entry: {processed_entry}")
-        openweather_historical_readings.append(processed_entry)
-
-            
+        url = f"https://api.openweathermap.org/data/3.0/onecall/timemachine?lat={lat}&lon={lon}&dt={int(datetime_input.timestamp())}&appid={OPENWEATHER_API_KEY}&units=metric"  
+        response = requests.get(url)  
+        response.raise_for_status()  
+        data = response.json()  
+
+        current_weather_data = data.get('data', [])[0] if data.get('data', []) else {}  
+        processed_entry = {  
+            'timestamp': datetime_input,  
+            'latitude': lat,  
+            'longitude': lon,  
+            'temperature': current_weather_data.get('temp'),  
+            'humidity': current_weather_data.get('humidity'),  
+            'wind_speed': current_weather_data.get('wind_speed')  
+        }  
+
+        # Append the processed entry to the session state list  
+        st.session_state.openweather_historical_readings.append(processed_entry)  
+
         return data  
-    except requests.exceptions.HTTPError as http_err:  
-        st.error(f"HTTP error occurred: {http_err}")  
+        # After fetching data  
+        logging.debug(f"Processed Entry: {processed_entry}")  
+        # When accessing historical data  
+        logging.debug(f"Historical data count: {len(openweather_historical_readings)}")  
     except Exception as err:  
-        st.error(f"An error occurred: {err}")  
+        st.error(f"An error occurred: {err}")    
         
 
 def fetch_openweather_forecast(lat, lon):  
@@ -166,9 +171,12 @@ def fetch_solunar_tide_data(lat, lon, datetime_input):
         st.error(f"Error fetching Solunar and Tide data: {e}")  
         return None
 
-def save_openweather_historical_to_csv(openweather_historical_readings):  
+def save_openweather_historical_to_csv():  
     """Save historical OpenWeather data to CSV for download."""  
     try:  
+        # Access the historical data from session state  
+        openweather_historical_readings = st.session_state.openweather_historical_readings  
+
         # Validate input data  
         if not openweather_historical_readings:  
             raise ValueError("No historical data available.")  
@@ -199,10 +207,12 @@ def save_openweather_historical_to_csv(openweather_historical_readings):
     
     except ValueError as ve:  
         logging.error(f"Validation error: {ve}")  
+        st.error(f"Validation error: {ve}")  
         return None  
     except Exception as e:  
         logging.error(f"Error saving historical data: {str(e)}")  
-        return None 
+        st.error(f"Error saving historical data: {str(e)}")  
+        return None  
   
 
 def save_stormglass_marine_to_csv():  
@@ -309,7 +319,7 @@ def main():
         
         # Download button for historical data  
         if st.button("Download Historical Weather Data"):  
-            csv_data = save_openweather_historical_to_csv(openweather_historical_readings)  
+            csv_data = save_openweather_historical_to_csv()  
     
             if csv_data:  
                 try:  
@@ -324,7 +334,7 @@ def main():
                 except Exception as e:  
                     st.error(f"Error preparing download: {str(e)}")  
             else:  
-                st.error("No historical data available...")  
+                    st.error("No historical data available.")    
  
 
         with col2: