Update app.py

app.py

@@ -95,21 +95,46 @@ def update_quiz_components(quiz_data):
         radio_updates.append(radio_update)
     return radio_updates + [gr.update(value="Please select answers and click 'Check Score'.", visible=True)]
 
-def collect_answers_and_calculate(*radio_values, /, quiz_data):  # / enforces quiz_data as positional-only after *radio_values
+# FIXED FUNCTION: Changed parameter signature to accept all arguments positionally
+def collect_answers_and_calculate(*all_inputs):
+    print(f"Total inputs received: {len(all_inputs)}")  # Debug print
+    
+    # The last input is quiz_data, the first 10 are radio values
+    radio_values = all_inputs[:10]  # First 10 inputs are radio button values
+    quiz_data = all_inputs[10]      # Last input is quiz_data
+    
     print(f"Received radio_values: {radio_values}")  # Debug print
     print(f"Received quiz_data: {quiz_data}")       # Debug print
-    user_answer_list = [answer for answer in radio_values if answer is not None]
+    
+    # Filter out None values but keep track of positions
+    user_answer_list = list(radio_values)
     correct_answers = [item.correct_answer for item in quiz_data.items[:10]]
+    
     print(f"User answers: {user_answer_list}")      # Debug print
     print(f"Correct answers: {correct_answers}")    # Debug print
-    score = sum(1 for u, c in zip(user_answer_list, correct_answers) if u == c)
-    print(f"Calculated score: {score}")             # Debug print
-    if score > 7:
-        message = f"### Excellent! You got {score} out of 10!"
-    elif score > 5:
-        message = f"### Good! You got {score} out of 10!"
+    
+    # Calculate score - only count answered questions
+    score = 0
+    answered_questions = 0
+    for u, c in zip(user_answer_list, correct_answers):
+        if u is not None:  # Only count if user answered
+            answered_questions += 1
+            if u == c:
+                score += 1
+    
+    print(f"Calculated score: {score}/{answered_questions}")  # Debug print
+    
+    if answered_questions == 0:
+        message = "### Please answer at least one question!"
+    elif score == answered_questions:
+        message = f"### Perfect! You got {score} out of {answered_questions} correct!"
+    elif score > answered_questions * 0.7:
+        message = f"### Excellent! You got {score} out of {answered_questions} correct!"
+    elif score > answered_questions * 0.5:
+        message = f"### Good! You got {score} out of {answered_questions} correct!"
     else:
-        message = f"### You got {score} out of 10! Don't worry. You can prepare well and try better next time!"
+        message = f"### You got {score} out of {answered_questions} correct! Don't worry. You can prepare well and try better next time!"
+    
     return message
 
 # Define a colorful theme
@@ -155,10 +180,10 @@ with gr.Blocks(title="Quiz Maker", theme=colorful_theme) as QUIZBOT:
         outputs=question_radios + [quiz_msg]
     )
 
-    # Register the click event for Check Score with explicit unpacking
+    # FIXED: Register the click event for Check Score with correct input handling
     check_score_btn.click(
         fn=collect_answers_and_calculate,
-        inputs=question_radios + [quiz_data_state],  # 10 radio values + 1 quiz_data
+        inputs=question_radios + [quiz_data_state],  # This creates a list of 11 inputs
         outputs=[score_output],
         api_name="check_score"
     )