Update src/app.py

src/app.py

@@ -612,7 +612,8 @@ with tab3:
                 valid_question_found = False
                 attempts = 0
                 
-                while not valid_question_found and attempts < 3:
+                # RETRY LOOP: Increased to 5 attempts to find a good chunk
+                while not valid_question_found and attempts < 5:
                     attempts += 1
                     q_ctx = quiz.get_document_context(st.session_state.username)
                     
@@ -623,8 +624,9 @@ with tab3:
                             300, model_choice, st.session_state.get("user_openai_key")
                         )
                         
-                        # Filter out skips or empty responses
-                        if "SKIP" not in question_text and len(question_text) > 10:
+                        # LOGIC UPDATE: Check for 'UNABLE' instead of 'SKIP'
+                        # We also check length to ensure we didn't get a blank response
+                        if "UNABLE" not in question_text and len(question_text) > 10:
                             valid_question_found = True
                             qs["active"] = True
                             qs["question_data"] = q_ctx
@@ -632,7 +634,7 @@ with tab3:
                             qs["feedback"] = None
                 
                 if not valid_question_found:
-                    st.warning("Tried to find a good engineering question but the available text was too administrative. Try uploading denser technical notes!")
+                    st.warning("Could not generate a question after 5 attempts. The selected documents might be too sparse or formatted as pure data tables.")
 
     # 2. AUTO-TRIGGER (Chained Question Logic)
     if st.session_state.quiz_trigger: