Update app.py

app.py

@@ -130,19 +130,6 @@ 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='%I:%M %p'):
-#     try:
-#         # 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 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:
-#         print(f"Error converting series to time: {e}")
-#         return series
-
 def safe_convert_to_time(series, format_str='%I:%M %p'):
     try:
         converted = pd.to_datetime(series, format='%H:%M:%S', errors='coerce')
@@ -382,17 +369,29 @@ def compute_student_metrics(df):
             'Engagement (%)': engagement_pct
         }
 
-        # # Store metrics with percentage sign
-        # student_metrics[student_name] = {
-        #     'Attendance (%)': f"{attendance_pct}%",  # Add percentage sign
-        #     'Engagement (%)': f"{engagement_pct}%"  # Add percentage sign
-        # }
-
     # Create a DataFrame from student_metrics
     student_metrics_df = pd.DataFrame.from_dict(student_metrics, orient='index').reset_index()
     student_metrics_df.rename(columns={'index': 'Student'}, inplace=True)
     return student_metrics_df
 
+def plot_student_metrics(student_metrics_df):
+    # Create a line graph for attendance and engagement
+    fig, ax = plt.subplots()
+
+    # Plotting Attendance and Engagement with specific colors
+    ax.plot(student_metrics_df['Student'], student_metrics_df['Attendance (%)'], marker='o', color='#005288', label='Attendance (%)')
+    ax.plot(student_metrics_df['Student'], student_metrics_df['Engagement (%)'], marker='o', color='#3AB0FF', label='Engagement (%)')
+
+    ax.set_xlabel('Student')
+    ax.set_ylabel('Percentage (%)')
+    ax.set_title('Student Attendance and Engagement Metrics')
+    ax.legend()
+    plt.xticks(rotation=45)
+
+    st.pyplot(fig)
+
+    return fig
+
 def plot_student_metrics(student_metrics_df):
     # Create the figure and axis
     fig, ax = plt.subplots(figsize=(10, 6))  # Increased figure size for better readability
@@ -432,7 +431,8 @@ def plot_student_metrics(student_metrics_df):
 
     # Display the plot
     plt.tight_layout()  # Adjust layout to fit elements
-    plt.show()  # Show the plot in a script environment (use st.pyplot(fig) in Streamlit)
+    # plt.show()  # Show the plot in a script environment (use st.pyplot(fig) in Streamlit)
+    st.pyplot(fig)  # This line displays the plot
 
     return fig