Update app.py

app.py

@@ -695,8 +695,9 @@ def process_video_file(video_file, analysis_types):
         temp_file.write(video_file.read())
         temp_video_path = temp_file.name
     
-    # Create a temp file for the output video
-    output_path = f"{temp_video_path}_processed.mp4"
+    # Create a temp file for the output video with more reliable path handling
+    # Use Path from pathlib to ensure proper path handling across platforms
+    output_path = str(Path(tempfile.gettempdir()) / f"{Path(temp_video_path).stem}_processed.mp4")
     
     # Open the video file
     cap = cv2.VideoCapture(temp_video_path)
@@ -925,16 +926,22 @@ def process_video_file(video_file, analysis_types):
         progress_bar.empty()
         status_text.empty()
         
+        # Verify the file exists before attempting to read it
+        if not os.path.exists(output_path):
+            raise FileNotFoundError(f"Processed video file not found at {output_path}")
+            
         # Read the processed video as bytes for download
         with open(output_path, 'rb') as file:
             processed_video_bytes = file.read()
         
         # Clean up temporary files
-        os.unlink(temp_video_path)
-        os.unlink(output_path)
+        try:
+            os.unlink(temp_video_path)
+            os.unlink(output_path)
+        except Exception as e:
+            st.warning(f"Warning: Could not delete temporary files: {str(e)}")
         
         # Return both the video and the detection statistics
-        processed_video_bytes = processed_video_bytes
         results = {"detection_stats": detection_stats}
         
         # Store results in session state for chatbot context
@@ -950,10 +957,15 @@ def process_video_file(video_file, analysis_types):
         cap.release()
         if 'out' in locals():
             out.release()
-        os.unlink(temp_video_path)
+        
+        # Only try to delete files if they exist
+        if os.path.exists(temp_video_path):
+            os.unlink(temp_video_path)
         if os.path.exists(output_path):
             os.unlink(output_path)
-        raise e
+            
+        # Raise the error with more context
+        raise Exception(f"Error processing video: {str(e)} (input: {temp_video_path}, output: {output_path})")
 
 def load_bigquery_table(dataset_id, table_id, limit=1000):
     """Load data directly from an existing BigQuery table"""