fix bugs

app/golf_swing_rag.py

@@ -210,21 +210,65 @@ class GolfSwingRAG:
         
         # Initialize OpenAI client using Streamlit secrets
         print("Initializing OpenAI client...")
+        openai_key = None
+        
+        # Try multiple methods to get the OpenAI API key
         try:
-            openai_key = st.secrets.get("openai", {}).get("api_key", "")
+            # Method 1: Direct access to OPENAI_API_KEY secret (most common for HF Spaces)
+            openai_key = st.secrets.get("OPENAI_API_KEY", "")
             if openai_key:
-                self.openai_client = openai.OpenAI(api_key=openai_key)
-                print("✓ OpenAI client initialized from Streamlit secrets")
+                print("✓ Found OpenAI key in st.secrets['OPENAI_API_KEY']")
             else:
-                print("No OpenAI key found in Streamlit secrets")
-        except (KeyError, FileNotFoundError, AttributeError) as e:
-            print(f"Error accessing Streamlit secrets: {e}")
-            # Fallback to environment variable if secrets not available
-            if os.getenv("OPENAI_API_KEY"):
-                self.openai_client = openai.OpenAI(api_key=os.getenv("OPENAI_API_KEY"))
-                print("✓ OpenAI client initialized from environment variable")
+                print("No key found in st.secrets['OPENAI_API_KEY']")
+        except Exception as e:
+            print(f"Error accessing st.secrets['OPENAI_API_KEY']: {e}")
+        
+        # Method 2: Try nested openai structure (fallback)
+        if not openai_key:
+            try:
+                openai_key = st.secrets.get("openai", {}).get("api_key", "")
+                if openai_key:
+                    print("✓ Found OpenAI key in nested st.secrets['openai']['api_key']")
+                else:
+                    print("No key found in st.secrets['openai']['api_key']")
+            except Exception as e:
+                print(f"Error accessing nested openai secrets: {e}")
+        
+        # Method 3: Environment variable fallback
+        if not openai_key:
+            openai_key = os.getenv("OPENAI_API_KEY", "")
+            if openai_key:
+                print("✓ Found OpenAI key in environment variable OPENAI_API_KEY")
             else:
-                print("No OpenAI API key found in environment variables either")
+                print("No key found in environment variable OPENAI_API_KEY")
+        
+        # Initialize client if we found a key
+        if openai_key and openai_key.startswith("sk-"):
+            try:
+                self.openai_client = openai.OpenAI(api_key=openai_key)
+                print("✓ OpenAI client initialized successfully")
+                
+                # Test the client with a simple request to verify it works
+                try:
+                    # Make a minimal test request
+                    test_response = self.openai_client.chat.completions.create(
+                        model="gpt-4o-mini",
+                        messages=[{"role": "user", "content": "Hi"}],
+                        max_tokens=5
+                    )
+                    print("✓ OpenAI client test successful")
+                except Exception as test_e:
+                    print(f"⚠️ OpenAI client test failed: {test_e}")
+                    self.openai_client = None
+                    
+            except Exception as e:
+                print(f"✗ Error initializing OpenAI client: {e}")
+                self.openai_client = None
+        else:
+            print("✗ No valid OpenAI API key found (should start with 'sk-')")
+            if openai_key:
+                print(f"Found key starting with: {openai_key[:10]}...")
+            self.openai_client = None
         
         print("=== End GolfSwingRAG Initialization Debug ===")
         print("")

app/streamlit_app.py

@@ -256,11 +256,12 @@ Performance Estimates:
     with col1:
         # Removed subheader
         
-        # Question input (removed label)
+        # Question input (with proper label)
         question = st.text_area(
-            "",  # Removed label
+            "Question",  # Proper label for accessibility
             height=100,
-            placeholder="Ask about your golf swing technique..."
+            placeholder="Ask about your golf swing technique...",
+            label_visibility="collapsed"  # Hide the label visually while keeping it for accessibility
         )
         
         # Removed settings section - using smart defaults instead
@@ -590,6 +591,32 @@ def main():
     st.title("Par-ity Project: Golf Swing Analysis 🏌️‍♀️")
     st.write("Founded to address the gender gap in golf participation and access to quality coaching resources, Par-ity Project is a technology-driven initiative empowering girls in golf through innovative AI based swing analysis. This technology uses computer vision and machine learning algorithms to analyze golf swings and provide personalized feedback to improve technique and performance.")
     
+    # Check and display OpenAI client status immediately
+    st.markdown("---")
+    st.subheader("🔧 System Status")
+    
+    if RAG_AVAILABLE:
+        try:
+            # Create a temporary RAG instance to check OpenAI client status
+            with st.spinner("Checking OpenAI API connection..."):
+                temp_rag = GolfSwingRAG()
+                
+            if temp_rag.openai_client:
+                st.success("✅ **OpenAI API Client**: Successfully initialized and tested")
+                st.info("💡 **ChatBot & AI Analysis**: Available - You can use the Golf Swing Chatbot and AI-powered improvement recommendations")
+            else:
+                st.error("❌ **OpenAI API Client**: Not available")
+                st.warning("⚠️ **Impact**: ChatBot will use fallback responses, AI analysis may be limited")
+                st.info("🔑 **Fix**: Check that your OPENAI_API_KEY is set correctly in Hugging Face Space secrets")
+        except Exception as e:
+            st.error(f"❌ **OpenAI API Client**: Error during initialization - {str(e)}")
+            st.warning("⚠️ **Impact**: ChatBot and AI features will use fallback mode")
+    else:
+        st.error("❌ **RAG System**: Not available due to missing dependencies")
+        st.warning("⚠️ **Impact**: ChatBot feature will be disabled")
+    
+    st.markdown("---")
+    
     # Initialize session state for storing analysis results
     if 'video_analyzed' not in st.session_state:
         st.session_state.video_analyzed = False