Update app.py

app.py

@@ -393,126 +393,49 @@ def compute_student_metrics(df):
     student_metrics_df.rename(columns={'index': 'Student'}, inplace=True)
     return student_metrics_df
 
-# 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 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([i + bar_width/2 for i in index], student_metrics_df['Engagement (%)'], 
-#                              width=bar_width, label='Engagement (%)', color='#3AB0FF', alpha=0.7)
-
-#     # 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}%', 
-#                 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)
-
-import matplotlib.pyplot as plt
 
 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
+    fig, ax = plt.subplots(figsize=(10, 6))  # Optional: Increased figure size for better readability
 
     # Width for the bars
-    bar_width = 0.35
-    index = range(len(student_metrics_df))
-    
-    # Plot Attendance and Engagement bars with transparency
-    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 * 0.7, label='Engagement (%)', color='#3AB0FF', alpha=0.7)  # Skinnier engagement bars
-    
+    bar_width = 0.35  # Width of the bars
+    index = range(len(student_metrics_df))  # Index for each student
+
+    # Plot Attendance and Engagement bars side by side
+    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([i + bar_width / 2 for i in index], 
+                              student_metrics_df['Engagement (%)'], 
+                              width=bar_width, label='Engagement (%)', 
+                              color='#3AB0FF', alpha=0.7)
+
     # Add labels to each bar
-    font_size = 10  # Font size for the labels
     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='white', fontsize=font_size)
+        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', fontsize=font_size)
-    
+        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, ha='right')  # Improved xticklabel alignment
-    
-    # Set y-axis limits from 0 to 100%
-    ax.set_ylim(0, 100)
+    ax.set_xticks(index)  # Set x-ticks to the index
+    ax.set_xticklabels(student_metrics_df['Student'], rotation=45, ha='right')  # Set student names as x-tick labels
 
-    # Tight layout to ensure all elements fit within the figure
-    fig.tight_layout()
-    
     # Display the plot
-    st.pyplot(fig)
-    
-    return fig
+    plt.tight_layout()  # Adjust layout to fit elements
+    plt.show()  # Show the plot in a script environment (use st.pyplot(fig) in Streamlit)
 
-# 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
+    return fig
 
 def download_chart(fig, filename):
     # Create a buffer to hold the image data