Update app.py

app.py

@@ -132,10 +132,11 @@ NOT_ENGAGED_STR = 'Not Engaged (less than 50%)'
 
 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
+        # Attempt to parse the time in 24-hour format
+        converted = pd.to_datetime(series, format='%H:%M:%S', errors='coerce').dt.time
+        
         if format_str:
-            # Format if a format string is provided
+            # Format the time to the specified format (hours and minutes only, with AM/PM)
             return pd.Series([time.strftime(format_str) if time is not None else None for time in converted])
         return converted
     except Exception as e:
@@ -384,13 +385,13 @@ def plot_student_metrics(student_metrics_df):
     bar_width = 0.4
     index = range(len(student_metrics_df))
 
-    # Plot Attendance and Engagement bars with different widths for overlapping
-    attendance_bars = ax.bar(index, student_metrics_df['Attendance (%)'], 
+    # Plot Attendance and Engagement bars side by side with transparency
+    attendance_bars = ax.bar([i - bar_width/2 for i in 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)
+    engagement_bars = ax.bar([i + bar_width/2 for i in index], student_metrics_df['Engagement (%)'], 
+                             width=bar_width, label='Engagement (%)', color='#3AB0FF', alpha=0.7)
 
-    # Add values to each bar
+    # Add labels 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}%', 
@@ -414,6 +415,44 @@ def plot_student_metrics(student_metrics_df):
 
     return fig
 
+# def plot_student_metrics(student_metrics_df):
+#     # Create the figure and axis
+#     fig, ax = plt.subplots()
+
+#     # Width for the bars and offset for overlapping effect
+#     bar_width = 0.4
+#     index = range(len(student_metrics_df))
+
+#     # 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')
+#     ax.set_ylabel('Percentage (%)')
+#     ax.set_title('Student Attendance and Engagement Metrics')
+#     ax.legend()
+#     ax.set_xticks(index)
+#     ax.set_xticklabels(student_metrics_df['Student'], rotation=45)
+
+#     # Display the plot
+#     st.pyplot(fig)
+
+#     return fig
+
 def download_chart(fig, filename):
     # Create a buffer to hold the image data
     buffer = io.BytesIO()