Updates

app.py

@@ -21,7 +21,7 @@ if 'module_correct_count' not in st.session_state:
 if 'module_question_count' not in st.session_state:
     st.session_state.module_question_count = {}
 if 'pdf_data' not in st.session_state:
-    st.session_state.pdf_data = b''  # Initialize as an empty byte string
+    st.session_state.pdf_data = None
 if 'selected_answer' not in st.session_state:
     st.session_state.selected_answer = None
 if 'button_label' not in st.session_state:
@@ -30,57 +30,54 @@ if 'start_time' not in st.session_state:
     st.session_state.start_time = time.time()
 
 def reset_pdf_cache():
-    st.session_state.pdf_data = b''  # Reset to an empty byte string
+    st.session_state.pdf_data = None
 
 def generate_pdf_report():
     pdf = FPDF()
     pdf.add_page()
-    pdf.set_font("Arial", style='B', size=12)
+    pdf.set_font("Arial", size=12)
     
-    # Header for PDF report
-    pdf.set_fill_color(211, 211, 211)  # Light gray
-    pdf.cell(0, 10, txt="🪄 Magic Math Quiz!", ln=True, align="C", fill=True)
-    pdf.ln(5)
-    pdf.set_font("Arial", size=8)
-    pdf.cell(0, 10, txt="by Ghassem Tofighi", ln=True, align="C", link="https://ghassem.com")
+    pdf.cell(200, 10, txt="Quiz Report", ln=True, align="C")
     pdf.ln(10)
 
-    for i, entry in enumerate(st.session_state.questions):
-        # Zebra background
-        if i % 2 == 0:
-            pdf.set_fill_color(245, 245, 245)  # Light gray
-        else:
-            pdf.set_fill_color(255, 255, 255)  # White
-
-        # Module Title
-        pdf.set_font("Arial", style='B', size=10)
-        pdf.multi_cell(0, 10, f"Module: {entry['module_title']}", border=1, fill=True)
-
-        # Question
-        pdf.set_font("Arial", style='B', size=10)
-        pdf.multi_cell(0, 10, f"Q{i+1}: {entry['question']}", border=1, fill=True)
-        
-        # Time Taken
-        pdf.set_font("Arial", size=10)
-        pdf.multi_cell(0, 10, f"Time Taken: {entry['time_taken']} seconds", border=1, fill=True)
-        
-        # Options
-        pdf.set_font("Arial", size=10)
-        options = ['a', 'b', 'c', 'd']
-        for j, option in enumerate(entry['options']):
-            if option == entry['correct_answer']:
-                pdf.set_text_color(0, 128, 0)  # Green for correct
-            elif option == entry['selected']:
-                pdf.set_text_color(255, 0, 0)  # Red for incorrect
-            else:
-                pdf.set_text_color(0, 0, 0)  # Default color for others
-
-            pdf.multi_cell(0, 10, f"{options[j]}. {option}", border=1, fill=True)
-        
-        pdf.set_text_color(0, 0, 0)  # Reset color
-        pdf.multi_cell(0, 10, f"Explanation: {entry['explanation']}", border=1, fill=True)
-        pdf.multi_cell(0, 10, f"Step-by-Step Solution: {', '.join(entry['step_by_step_solution'])}", border=1, fill=True)
-        pdf.ln(10)
+    for module, data in st.session_state.module_question_count.items():
+        correct_count = st.session_state.module_correct_count.get(module, 0)
+        total_count = data
+        pdf.cell(200, 10, txt=f"Module: {module}", ln=True, align="L")
+        pdf.ln(5)
+        pdf.cell(200, 10, txt=f"Correct Answers: {correct_count}/{total_count}", ln=True, align="L")
+        pdf.ln(5)
+
+        for entry in st.session_state.questions:
+            if entry['module'] == module:
+                question, options, selected, correct, explanation, step_by_step_solution = (
+                    entry['question'], 
+                    entry['options'], 
+                    entry['selected'] if entry['selected'] is not None else "Not Answered",
+                    entry['correct_answer'], 
+                    entry['explanation'],
+                    entry['step_by_step_solution']
+                )
+                pdf.multi_cell(0, 10, f"Q: {question}")
+                for option in options:
+                    if option == correct:
+                        pdf.set_text_color(0, 128, 0)  # Green for correct
+                        pdf.multi_cell(0, 10, f"{option}")
+                    elif option == selected:
+                        pdf.set_text_color(255, 0, 0)  # Red for incorrect
+                        pdf.multi_cell(0, 10, f"{option}")
+                    else:
+                        pdf.set_text_color(0, 0, 0)  # Default color for others
+                        pdf.multi_cell(0, 10, f"   {option}")
+                pdf.set_text_color(0, 0, 0)  # Reset color
+                pdf.multi_cell(0, 10, f"Explanation: {explanation}")
+                pdf.ln(5)
+                pdf.multi_cell(0, 10, "Step-by-Step Solution:")
+                for step in step_by_step_solution:
+                    pdf.multi_cell(0, 10, step)
+                pdf.ln(10)
+
+        pdf.ln(10)  # Add space after each module
 
     return pdf.output(dest='S').encode('latin1', 'replace')