Update app.py

app.py

@@ -7,14 +7,18 @@ import feedparser
 
 load_dotenv()
 
+# Load API keys from environment variables
 ORS_API_KEY = os.getenv("ORS_API_KEY")
 OPENWEATHER_API_KEY = os.getenv("OPENWEATHER_API_KEY")
 
+# Initialize geolocator
 geolocator = Nominatim(user_agent="commute_planner")
 
+# Streamlit UI
 st.title("🚗 Free Commute Planner")
 st.markdown("Estimate travel time, check weather, AQI, and traffic-related news — no credit card needed!")
 
+# Inputs
 source = st.text_input("Enter your current location:")
 destination = st.text_input("Enter your destination:")
 
@@ -34,7 +38,7 @@ if st.button("Plan My Commute") and source and destination:
         if not src_coords or not dest_coords:
             st.error("Could not find coordinates. Please enter valid locations.")
         else:
-            # OpenRouteService: Travel Time
+            # ✅ Travel Time via OpenRouteService
             ors_url = "https://api.openrouteservice.org/v2/directions/driving-car"
             headers = {
                 "Authorization": ORS_API_KEY,
@@ -47,15 +51,25 @@ if st.button("Plan My Commute") and source and destination:
                 ]
             }
             response = requests.post(ors_url, json=data, headers=headers)
+
             if response.status_code == 200:
                 result = response.json()
-                duration_sec = result['features'][0]['properties']['segments'][0]['duration']
-                duration_min = round(duration_sec / 60, 2)
-                st.success(f"🕒 Estimated Travel Time: {duration_min} minutes")
+                try:
+                    duration_sec = result['features'][0]['properties']['segments'][0]['duration']
+                    duration_min = round(duration_sec / 60, 2)
+                    st.success(f"🕒 Estimated Travel Time: {duration_min} minutes")
+                except (KeyError, IndexError):
+                    st.error("Received a response but couldn't parse travel time.")
+                    st.markdown("**Response from OpenRouteService:**")
+                    st.json(result)
             else:
-                st.warning("Could not retrieve travel time. Please check your API key and inputs.")
+                st.error(f"OpenRouteService API error: {response.status_code}")
+                try:
+                    st.json(response.json())
+                except:
+                    st.write(response.text)
 
-            # OpenWeatherMap: Temperature
+            # ✅ Weather from OpenWeatherMap
             weather_url = "https://api.openweathermap.org/data/2.5/weather"
             weather_params = {
                 "lat": src_coords[0],
@@ -67,7 +81,7 @@ if st.button("Plan My Commute") and source and destination:
             temp = weather.get("main", {}).get("temp", "N/A")
             st.info(f"🌡️ Current Temperature: {temp}°C")
 
-            # OpenWeatherMap: AQI
+            # ✅ Air Quality Index from OpenWeatherMap
             aqi_url = "https://api.openweathermap.org/data/2.5/air_pollution"
             aqi_params = {
                 "lat": src_coords[0],
@@ -85,14 +99,16 @@ if st.button("Plan My Commute") and source and destination:
             }
             st.info(f"🌀 Air Quality Index: {aqi} - {aqi_levels.get(aqi, 'Unknown')}")
 
-            # Traffic News using RSS
+            # ✅ Traffic News via RSS Feed
             st.subheader("📰 Traffic-Related News")
-            feed_url = "https://www.rssmix.com/u/17853882/rss.xml"  # Replace with a real local traffic feed or use another from your city
-            feed = feedparser.parse(feed_url)
+            rss_url = "https://www.rssmix.com/u/17853882/rss.xml"  # Replace with your preferred traffic-related feed
+            feed = feedparser.parse(rss_url)
+
             count = 0
             for entry in feed.entries:
-                if "traffic" in entry.title.lower() or "accident" in entry.title.lower() or "road" in entry.title.lower():
+                if any(keyword in entry.title.lower() for keyword in ["traffic", "accident", "road", "construction"]):
                     st.markdown(f"[{entry.title}]({entry.link})")
                     count += 1
+
             if count == 0:
                 st.write("No recent traffic-related updates found.")