Updates

app.py

@@ -1,12 +1,8 @@
 import streamlit as st
 import importlib
-import logging
 from fpdf import FPDF
 import uuid
 
-# Configure logging
-logging.basicConfig(level=logging.INFO)
-
 # List of available modules with shorter names and icons
 module_names = {
     "Bases": "presentation_bases",
@@ -48,8 +44,6 @@ if 'module_correct_count' not in st.session_state:
     st.session_state.module_correct_count = {name: 0 for name in module_names}
 if 'module_question_count' not in st.session_state:
     st.session_state.module_question_count = {name: 0 for name in module_names}
-if 'selected_answer' not in st.session_state:
-    st.session_state.selected_answer = None
 if 'pdf_data' not in st.session_state:
     st.session_state.pdf_data = None
 
@@ -78,7 +72,7 @@ def generate_pdf_report():
                 question, options, selected, correct, explanation, step_by_step_solution = (
                     entry['question'], 
                     entry['options'], 
-                    entry['selected'] if entry['selected'] is not None else "Not Answered",  # Handle case where selected is None
+                    entry['selected'] if entry['selected'] is not None else "Not Answered",
                     entry['correct_answer'], 
                     entry['explanation'],
                     entry['step_by_step_solution']
@@ -118,6 +112,9 @@ def generate_new_question(module_name):
     question_data['answered'] = False
     question_data['module'] = module_name  # Add the module name to the question data
     question_data['selected'] = None  # Initialize 'selected' to None
+    # Ensure there are 5 options
+    if len(question_data['options']) != 5:
+        st.warning(f"Question does not have 5 options. Found {len(question_data['options'])}.")
     return question_data
 
 def navigate_question(direction):
@@ -142,23 +139,33 @@ if module_name != st.session_state.current_module:
     st.session_state.questions = [generate_new_question(module_name)]
 
 # Load the current module's question
-current_question = st.session_state.questions[st.session_state.current_index]
+if st.session_state.current_index < len(st.session_state.questions):
+    current_question = st.session_state.questions[st.session_state.current_index]
+else:
+    current_question = generate_new_question(st.session_state.current_module)
+    st.session_state.questions.append(current_question)
+    st.session_state.current_index = len(st.session_state.questions) - 1
 
 # Display module title and description
 st.title(module_name)
 st.write(module_descriptions.get(module_name, "No description available for this module."))
 st.write(current_question["question"])
 
-# Render options without pre-selection
-if not current_question.get('answered', False):
+# Use a form to prevent rerun on option selection
+with st.form(key=f'question_form_{st.session_state.current_index}'):
     selected_answer = st.radio(
         "Choose an answer:", 
         options=current_question['options'], 
-        key=f"question_{st.session_state.current_index}",
+        key=f"question_{st.session_state.current_index}_options",
         index=None  # No pre-selection
     )
+    submit_button = st.form_submit_button(label="Submit")
 
-    if st.button("Submit"):
+if submit_button and not current_question.get('answered', False):
+    if selected_answer is None:
+        st.warning("Please select an answer before submitting.")
+    else:
+        # Process the answer
         current_question['selected'] = selected_answer
         current_question['answered'] = True
         st.session_state.module_question_count[module_name] += 1
@@ -176,7 +183,7 @@ if current_question.get('answered', False):
             st.markdown(f"<span style='color:red;'>{option} ❌</span>", unsafe_allow_html=True)
         else:
             st.markdown(f"{option}")
-
+    
     # Show explanation and step-by-step solution
     st.write(f"**Explanation:** {current_question['explanation']}")
     st.write("**Step-by-Step Solution:**")
@@ -186,11 +193,11 @@ if current_question.get('answered', False):
 # Navigation buttons (placed after question and options)
 col1, col2 = st.columns([1, 1])
 with col1:
-    if st.button("⬅️ Prev", disabled=st.session_state.current_index == 0):
+    if st.button("⬅️ Prev", key=f'prev_button_{st.session_state.current_index}', disabled=st.session_state.current_index == 0):
         navigate_question("prev")
 
 with col2:
-    if st.button("➡️ Next"):
+    if st.button("➡️ Next", key=f'next_button_{st.session_state.current_index}'):
         navigate_question("next")
 
 # PDF Download Button (at the bottom)