remove debug and fix array of 0

README.md

@@ -4,7 +4,7 @@ emoji: ⚡
 colorFrom: red
 colorTo: pink
 sdk: streamlit
-sdk_version: 1.26.0
+sdk_version: 1.24.1
 app_file: app.py
 pinned: false
 ---

app.py

@@ -47,7 +47,7 @@ if selected_shoes:
     })
     st.bar_chart(chart_data)
 else:
-    st.write("No column(s) selected")
+    st.info("No column(s) selected")
 
 # get activities on a period
 st.divider()
@@ -65,42 +65,48 @@ if st.session_state.start_date > st.session_state.end_date:
 else:
     st.session_state.activities = strava.get_activities_on_period(strava_auth, [], st.session_state.start_date, st.session_state.end_date, 1)
 
-st.write(f"You got {len(st.session_state.activities)} activities")
-
-activities_zones = {}
-for activity in st.session_state.activities:
-    if not activity["has_heartrate"]:
-        continue
-    try:
-        st.session_state.activity_zones = strava.get_activity_zones(strava_auth, activity["id"])[0]["distribution_buckets"]
-    except Exception as e:
-        st.write(e)
-    if not activities_zones:
-        activities_zones = {idx: zone["time"] // 60 for idx, zone in enumerate(st.session_state.activity_zones)}
-    else:
-        for idx, zone in enumerate(st.session_state.activity_zones):
-            activities_zones[idx] += (zone["time"] // 60)
-
-zones_label = ["zone 1", "zone 2", "zone 3", "zone 4", "zone 5"]
-zones_df = pd.DataFrame({
-    'zones': zones_label,
-    'minutes': activities_zones.values()
-})
-
-scale = alt.Scale(
-    domain=zones_label,
-    range=["#008000", "#ffcf3e", "#f67200", "#ee1010", "#3f2204"],
-)
-color = alt.Color("zones:N", scale=scale)
-
-bars = (
-    alt.Chart(zones_df)
-    .mark_bar()
-    .encode(
-        x="zones",
-        y="minutes",
-        color=color,
+if len(st.session_state.activities) == 0:
+    st.error("No activities found on this period")
+    st.stop()
+else:
+    activities_zones = {}
+    number_activities_with_heartrate = 0
+    for activity in st.session_state.activities:
+        if not activity["has_heartrate"]:
+            continue
+        try:
+            st.session_state.activity_zones = strava.get_activity_zones(strava_auth, activity["id"])[0]["distribution_buckets"]
+            number_activities_with_heartrate += 1
+        except Exception as e:
+            st.write(e)
+        if not activities_zones:
+            activities_zones = {idx: zone["time"] // 60 for idx, zone in enumerate(st.session_state.activity_zones)}
+        else:
+            for idx, zone in enumerate(st.session_state.activity_zones):
+                activities_zones[idx] += (zone["time"] // 60)
+
+    st.info(f"You got {len(st.session_state.activities)} activities on this period and {number_activities_with_heartrate} have heart rate information.")
+
+    zones_label = ["zone 1", "zone 2", "zone 3", "zone 4", "zone 5"]
+    zones_df = pd.DataFrame({
+        'zones': zones_label,
+        'minutes': activities_zones.values()
+    })
+
+    scale = alt.Scale(
+        domain=zones_label,
+        range=["#008000", "#ffcf3e", "#f67200", "#ee1010", "#3f2204"],
+    )
+    color = alt.Color("zones:N", scale=scale)
+
+    bars = (
+        alt.Chart(zones_df)
+        .mark_bar()
+        .encode(
+            x="zones",
+            y="minutes",
+            color=color,
+        )
     )
-)
 
-st.altair_chart(bars, theme="streamlit", use_container_width=True)
+    st.altair_chart(bars, theme="streamlit", use_container_width=True)

requirements.txt

@@ -1,4 +1,4 @@
-streamlit==1.26.0
+streamlit==1.24.1
 httpx==0.24.1
 pandas==2.1.0
 python-dotenv==1.0.0

strava.py

@@ -9,8 +9,6 @@ from datetime import datetime
 load_dotenv()
 
 APP_URL = os.environ["APP_URL"]
-print(os.environ)
-print(st.secrets)
 STRAVA_CLIENT_ID = os.environ["STRAVA_CLIENT_ID"]
 STRAVA_CLIENT_SECRET = os.environ["STRAVA_CLIENT_SECRET"]
 STRAVA_AUTHORIZATION_URL = "https://www.strava.com/oauth/authorize"