Update app.py

app.py

@@ -261,7 +261,6 @@ def generate_feedback_from_prompt(prompt_input: str) -> str:
     
     try:
         print("Generating feedback with Gemini...")
-        print(f"Prompt length: {len(prompt_input)} characters")
         
         # Extract just the user's input text
         user_text = prompt_input
@@ -270,70 +269,43 @@ def generate_feedback_from_prompt(prompt_input: str) -> str:
             if len(parts) > 1:
                 user_text = parts[1].strip().strip('"')
         
-        # Simplified prompt
-        simple_prompt = f"""Provide educational feedback on this architectural description in 250-300 words:
-
-{user_text}
-
-Be helpful and constructive."""
-        
-        print(f"Simplified prompt length: {len(simple_prompt)} characters")
-        
-        generation_config = {
-            'temperature': 0.7,
-            'max_output_tokens': 600,
-            'top_p': 0.95,
-        }
+        # Ultra-simplified prompt - just the core request
+        simple_prompt = f"Provide brief educational feedback on this architectural description: {user_text}"
         
-        response = gemini_model.generate_content(
-            simple_prompt,
-            generation_config=generation_config
-        )
+        print(f"Sending prompt ({len(simple_prompt)} chars)")
         
-        print(f"Response received. Candidates: {len(response.candidates) if response.candidates else 0}")
+        # Minimal configuration - only what's absolutely necessary
+        response = gemini_model.generate_content(simple_prompt)
         
-        if response.candidates:
-            candidate = response.candidates[0]
-            print(f"Finish reason: {candidate.finish_reason}")
-            print(f"Has content: {candidate.content is not None}")
-            if candidate.content:
-                print(f"Parts count: {len(candidate.content.parts) if candidate.content.parts else 0}")
-                if candidate.safety_ratings:
-                    ratings_info = [(r.category, r.probability) for r in candidate.safety_ratings]
-                    print(f"Safety ratings: {ratings_info}")
+        print(f"Response received")
         
         # Extract text
         llm_response_text = None
         
         try:
             llm_response_text = response.text
-            print("✓ Got text from response.text")
+            print(f"✓ Got text ({len(llm_response_text)} chars)")
         except Exception as e:
-            print(f"Failed to get response.text: {e}")
+            print(f"Failed to get text: {str(e)[:100]}")
             
+            # Try alternative extraction
             if response.candidates and response.candidates[0].content:
                 candidate = response.candidates[0]
                 if candidate.content.parts:
                     texts = [part.text for part in candidate.content.parts if hasattr(part, 'text')]
                     if texts:
                         llm_response_text = "".join(texts)
-                        print("✓ Got text from candidate parts")
+                        print(f"✓ Got text from parts ({len(llm_response_text)} chars)")
         
         if not llm_response_text:
-            if response.candidates:
-                finish_reason = response.candidates[0].finish_reason
-                if finish_reason == 2:
-                    return "⚠️ Content blocked by safety filter. This may be due to API restrictions. Try:\n1. Using simpler, more direct language\n2. Avoiding technical jargon\n3. Shortening your description"
-            return "⚠️ No response generated. Please try again with simpler input."
+            return "⚠️ No response generated. This may be an API limitation. Try:\n- Shorter, simpler descriptions\n- Removing technical terms\n- Testing with basic input like 'large windows'"
         
-        llm_response_text = llm_response_text.strip()
-        print(f"✓ Feedback generated ({len(llm_response_text)} chars)")
-        return llm_response_text
+        return llm_response_text.strip()
         
     except Exception as e:
         error_msg = str(e)
-        print(f"❌ Full error: {error_msg}")
-        return f"❌ Error: {error_msg}"
+        print(f"Error: {error_msg[:200]}")
+        return f"Error: {error_msg}"
 
 # ============================================
 # 9. GRADIO INTERFACE