Update app.py

app.py

@@ -130,7 +130,7 @@ ENGAGED_STR = 'Engaged (Respect, Responsibility, Effort)'
 PARTIALLY_ENGAGED_STR = 'Partially Engaged (about 50%)'
 NOT_ENGAGED_STR = 'Not Engaged (less than 50%)'
 
-def safe_convert_to_time(series, format_str=None):
+def safe_convert_to_time(series, format_str='%I:%M %p'):
     try:
         # Attempt to convert to time, ignoring errors
         converted = pd.to_datetime(series, errors='coerce').dt.time
@@ -384,11 +384,22 @@ def plot_student_metrics(student_metrics_df):
     bar_width = 0.4
     index = range(len(student_metrics_df))
 
-    # Plot Attendance and Engagement bars side by side with transparency
-    ax.bar([i - bar_width/2 for i in index], student_metrics_df['Attendance (%)'], 
-           width=bar_width, label='Attendance (%)', color='#005288', alpha=0.7)
-    ax.bar([i + bar_width/2 for i in index], student_metrics_df['Engagement (%)'], 
-           width=bar_width, label='Engagement (%)', color='#3AB0FF', alpha=0.7)
+    # Plot Attendance and Engagement bars with different widths for overlapping
+    attendance_bars = ax.bar(index, student_metrics_df['Attendance (%)'], 
+                             width=bar_width, label='Attendance (%)', color='#005288', alpha=0.7)
+    engagement_bars = ax.bar(index, student_metrics_df['Engagement (%)'], 
+                             width=bar_width / 2, label='Engagement (%)', color='#3AB0FF', alpha=0.7)
+
+    # Add values to each bar
+    for bar in attendance_bars:
+        height = bar.get_height()
+        ax.text(bar.get_x() + bar.get_width() / 2, height, f'{height:.1f}%', 
+                ha='center', va='bottom', color='black')
+
+    for bar in engagement_bars:
+        height = bar.get_height()
+        ax.text(bar.get_x() + bar.get_width() / 2, height, f'{height:.1f}%', 
+                ha='center', va='bottom', color='black')
 
     # Set labels, title, and legend
     ax.set_xlabel('Student')