Update app.py

app.py

@@ -26,11 +26,86 @@ model = genai.GenerativeModel('gemini-pro')
 # Streamlit app
 st.set_page_config(page_title="Strava Run Analysis", layout="wide")
 
-# (Custom CSS remains the same)
+# Custom CSS for better UI
+st.markdown("""
+<style>
+    .stApp {
+        background-color: #f0f2f6;
+    }
+    .stButton>button {
+        background-color: #fc4c02;
+        color: white;
+        font-weight: bold;
+        border-radius: 5px;
+        border: none;
+        padding: 0.5rem 1rem;
+    }
+    .stButton>button:hover {
+        background-color: #e34402;
+    }
+    .stSelectbox {
+        color: #333333;
+    }
+    .stPlotlyChart {
+        background-color: white;
+        border-radius: 10px;
+        box-shadow: 0 4px 6px rgba(0,0,0,0.1);
+        padding: 1rem;
+        margin-bottom: 2rem;
+    }
+    .stat-card {
+        background-color: white;
+        border-radius: 10px;
+        box-shadow: 0 4px 6px rgba(0,0,0,0.1);
+        padding: 1.5rem;
+        text-align: center;
+    }
+    .stat-card h3 {
+        color: #fc4c02;
+        font-size: 2.5rem;
+        margin-bottom: 0.5rem;
+    }
+    .stat-card p {
+        color: #666;
+        font-size: 1rem;
+        margin: 0;
+    }
+    .section-header {
+        color: #333;
+        font-size: 1.8rem;
+        font-weight: bold;
+        margin-top: 2rem;
+        margin-bottom: 1rem;
+    }
+</style>
+""", unsafe_allow_html=True)
 
 st.title("🏃‍♂️ Strava Run Analysis")
 
-# (Strava authentication remains the same)
+# Strava manual authentication
+if 'access_token' not in st.session_state:
+    st.write("To use this app, you need to authorize it with Strava. Follow these steps:")
+    st.write("1. Click the button below to go to Strava's authorization page:")
+    auth_url = f"https://www.strava.com/oauth/authorize?client_id={STRAVA_CLIENT_ID}&response_type=code&redirect_uri=http://localhost&approval_prompt=force&scope=read_all,profile:read_all,activity:read_all"
+    st.markdown(f"<a href='{auth_url}' target='_blank'><button style='background-color: #fc4c02; color: white; padding: 0.5rem 1rem; border: none; border-radius: 5px; cursor: pointer;'>Authorize Strava</button></a>", unsafe_allow_html=True)
+    st.write("2. Log in to Strava if needed and click 'Authorize'")
+    st.write("3. After authorizing, you'll be redirected to a page that may show an error. This is expected!")
+    st.write("4. Copy the 'code' parameter from the URL of that page.")
+    st.write("5. Paste that code below:")
+    
+    auth_code = st.text_input("Paste the authorization code here:")
+    if auth_code:
+        try:
+            token_response = client.exchange_code_for_token(
+                client_id=STRAVA_CLIENT_ID,
+                client_secret=STRAVA_CLIENT_SECRET,
+                code=auth_code
+            )
+            st.session_state.access_token = token_response['access_token']
+            st.success("Authorization successful! Refreshing the app...")
+            st.rerun()
+        except Exception as e:
+            st.error(f"An error occurred: {str(e)}. Please try authorizing again.")
 
 if 'access_token' in st.session_state:
     client.access_token = st.session_state.access_token
@@ -51,20 +126,17 @@ if 'access_token' in st.session_state:
         df = pd.DataFrame([{
             'name': run.name,
             'distance': float(run.distance) / 1000,  # Convert to km
-            'moving_time': run.moving_time.total_seconds() / 60 if run.moving_time else None,  # Convert to minutes, handle None
-            'total_elevation_gain': float(run.total_elevation_gain) if run.total_elevation_gain else 0,
-            'average_speed': float(run.average_speed) * 3.6 if run.average_speed else None,  # Convert to km/h
+            'moving_time': run.moving_time.total_seconds() / 60 if run.moving_time else None,  # Convert to minutes
+            'total_elevation_gain': float(run.total_elevation_gain),
+            'average_speed': float(run.average_speed) * 3.6,  # Convert to km/h
             'average_heartrate': float(run.average_heartrate) if run.average_heartrate else None,
-            'start_date': run.start_date.replace(tzinfo=None) if run.start_date else None
+            'start_date': run.start_date.replace(tzinfo=None)
         } for run in runs])
-        df['pace'] = df['moving_time'] / df['distance'] if df['moving_time'] is not None else None  # Calculate pace (min/km)
+        df['pace'] = df['moving_time'] / df['distance']  # Calculate pace (min/km)
         return df
 
     try:
         df = fetch_run_activities()
-        if df.empty:
-            st.warning("No running activities found. Make sure you have some runs in your Strava account.")
-            st.stop()
     except Exception as e:
         st.error(f"Error fetching activities: {str(e)}. Please try again later.")
         st.stop()
@@ -79,10 +151,7 @@ if 'access_token' in st.session_state:
         st.markdown("<div class='stat-card'><h3>{:.0f} km</h3><p>Total Distance</p></div>".format(df['distance'].sum()), unsafe_allow_html=True)
     with col3:
         total_time = df['moving_time'].sum()
-        if pd.notna(total_time):
-            st.markdown("<div class='stat-card'><h3>{:.0f}h {:.0f}m</h3><p>Total Time</p></div>".format(total_time // 60, total_time % 60), unsafe_allow_html=True)
-        else:
-            st.markdown("<div class='stat-card'><h3>N/A</h3><p>Total Time</p></div>", unsafe_allow_html=True)
+        st.markdown("<div class='stat-card'><h3>{:.0f}h {:.0f}m</h3><p>Total Time</p></div>".format(total_time // 60, total_time % 60), unsafe_allow_html=True)
     with col4:
         st.markdown("<div class='stat-card'><h3>{:.0f} m</h3><p>Total Elevation Gain</p></div>".format(df['total_elevation_gain'].sum()), unsafe_allow_html=True)
 
@@ -90,7 +159,6 @@ if 'access_token' in st.session_state:
     st.markdown("<h2 class='section-header'>📈 Run Analysis</h2>", unsafe_allow_html=True)
     
     # Weekly distance
-    df['start_date'] = pd.to_datetime(df['start_date'])
     weekly_distance = df.resample('W', on='start_date')['distance'].sum().reset_index()
     fig_weekly = px.bar(weekly_distance, x='start_date', y='distance', 
                         title="Weekly Running Distance",
@@ -99,17 +167,14 @@ if 'access_token' in st.session_state:
     st.plotly_chart(fig_weekly, use_container_width=True)
 
     # Pace improvement
-    if df['pace'].notna().any():
-        df_sorted = df.sort_values('start_date')
-        fig_pace = px.scatter(df_sorted, x='start_date', y='pace', 
-                              title="Running Pace Over Time",
-                              labels={'pace': 'Pace (min/km)', 'start_date': 'Date'})
-        fig_pace.add_trace(go.Scatter(x=df_sorted['start_date'], y=df_sorted['pace'].rolling(window=10).mean(),
-                                      mode='lines', name='10-run moving average'))
-        fig_pace.update_layout(yaxis_title="Pace (min/km)")
-        st.plotly_chart(fig_pace, use_container_width=True)
-    else:
-        st.info("No pace data available for analysis.")
+    df_sorted = df.sort_values('start_date')
+    fig_pace = px.scatter(df_sorted, x='start_date', y='pace', 
+                          title="Running Pace Over Time",
+                          labels={'pace': 'Pace (min/km)', 'start_date': 'Date'})
+    fig_pace.add_trace(go.Scatter(x=df_sorted['start_date'], y=df_sorted['pace'].rolling(window=10).mean(),
+                                  mode='lines', name='10-run moving average'))
+    fig_pace.update_layout(yaxis_title="Pace (min/km)")
+    st.plotly_chart(fig_pace, use_container_width=True)
 
     # Heart rate improvement (if data available)
     if df['average_heartrate'].notna().any():