Update app.py

app.py

@@ -332,28 +332,28 @@ def update_gallery():
                 st.markdown(get_download_link(file, "image/png", "Download Image"), unsafe_allow_html=True)
 
 def get_available_video_devices():
-    video_devices = []
-    # Try V4L2 explicitly, then fall back to default backend
-    for i in range(10):
-        cap = cv2.VideoCapture(i, cv2.CAP_V4L2)
-        if not cap.isOpened():
-            cap = cv2.VideoCapture(i)  # Try default backend
-        if cap.isOpened():
-            video_devices.append(f"Camera {i}")
-            logger.info(f"Detected camera at index {i}")
-            cap.release()
-        else:
-            logger.debug(f"No camera detected at index {i}")
-    if not video_devices:
-        logger.warning("No cameras detected by OpenCV. Checking system devices...")
-        # Fallback: Check /dev/video* existence (Linux-specific)
-        video_nodes = glob.glob("/dev/video*")
-        if video_nodes:
-            video_devices = [f"Camera {i}" for i in range(len(video_nodes))]
-            logger.info(f"Found video nodes: {video_devices}")
+    # Placeholder list since we can't directly query browser devices from Python
+    # This is a fallback; ideally, use a custom component for navigator.mediaDevices.enumerateDevices()
+    video_devices = ["Camera 0", "Camera 1"]  # Default assumption
+    try:
+        # Attempt OpenCV detection as a secondary check
+        detected = []
+        for i in range(10):
+            cap = cv2.VideoCapture(i, cv2.CAP_V4L2)
+            if not cap.isOpened():
+                cap = cv2.VideoCapture(i)  # Try default backend
+            if cap.isOpened():
+                detected.append(f"Camera {i}")
+                logger.info(f"Detected camera at index {i}")
+                cap.release()
+            else:
+                logger.debug(f"No camera detected at index {i}")
+        if detected:
+            video_devices = detected
         else:
-            video_devices = ["Camera 0", "Camera 1"]  # Default fallback for UI
-            logger.warning("No /dev/video* nodes found; using default Camera 0 and 1")
+            logger.warning("OpenCV detected no cameras; using browser-inferred defaults")
+    except Exception as e:
+        logger.error(f"Error detecting video devices with OpenCV: {str(e)}")
     return video_devices
 
 def mock_search(query: str) -> str:
@@ -552,19 +552,15 @@ with tab2:
     st.header("Camera Snap 📷 (Dual Capture!)")
     video_devices = get_available_video_devices()
     st.subheader("Camera Settings ⚙️")
-    if not any("Camera" in dev for dev in video_devices):
-        st.warning("No video devices detected by OpenCV! 📷 Browser capture still active. Check logs and permissions (e.g., 'sudo usermod -aG video $USER').")
-    else:
-        st.write(f"Detected Video Devices: {', '.join(video_devices)}")
-    
+    st.write(f"Available Cameras: {', '.join(video_devices)}")
+    st.info("Camera selection is informational. Use your browser settings to switch cameras (e.g., Chrome > Settings > Privacy > Camera).")
+
     # Camera selection with defaults
-    default_cam0_index = 0 if video_devices else 0  # Default to Camera 0 or first fallback
-    default_cam1_index = 1 if len(video_devices) > 1 else 0  # Default to Camera 1 or same as Cam 0
+    default_cam0_index = 0
+    default_cam1_index = 1 if len(video_devices) > 1 else 0
     selected_cam0 = st.selectbox("Select Camera 0", video_devices, index=default_cam0_index, key="cam0_select")
     selected_cam1 = st.selectbox("Select Camera 1", video_devices, index=default_cam1_index, key="cam1_select")
     
-    st.info("Note: Camera selection is informational. Actual device used depends on browser settings. Configure in browser (e.g., Chrome > Settings > Privacy > Camera).")
-    
     slice_count = st.number_input("Image Slice Count 🎞️", min_value=1, max_value=20, value=10, help="How many snaps to dream of? (Automation’s on vacation! 😜)")
     video_length = st.number_input("Video Dream Length (seconds) 🎥", min_value=1, max_value=30, value=10, help="Imagine a vid this long—sadly, we’re stuck with pics for now! 😂")