Update app.py

app.py

@@ -8,7 +8,7 @@ from ultralytics import YOLO
 import requests
 
 ###############################################################################
-# Helper: Embed an HTML5 video for autoplay (muted) display.
+# Helper: Embed an HTML5 video that autoplays (muted) with controls.
 ###############################################################################
 def show_autoplay_video(video_bytes: bytes, title: str = "Video"):
     if not video_bytes:
@@ -25,7 +25,7 @@ def show_autoplay_video(video_bytes: bytes, title: str = "Video"):
     st.markdown(video_html, unsafe_allow_html=True)
 
 ###############################################################################
-# Session state initialization for processed results.
+# Session state initialization (for processed results)
 ###############################################################################
 if "processed_frames" not in st.session_state:
     st.session_state["processed_frames"] = []
@@ -35,7 +35,7 @@ if "shortened_video_ready" not in st.session_state:
     st.session_state["shortened_video_ready"] = False
 
 ###############################################################################
-# Configure YOLO model path and page layout.
+# Configure YOLO model path and page layout
 ###############################################################################
 model_path = 'https://huggingface.co/spaces/tstone87/ccr-colorado/resolve/main/best.pt'
 st.set_page_config(
@@ -46,7 +46,7 @@ st.set_page_config(
 )
 
 ###############################################################################
-# SIDEBAR: Video input options, confidence, sampling options, and example selection.
+# SIDEBAR: Video input options, confidence, sampling options, and example selection
 ###############################################################################
 with st.sidebar:
     st.header("Video Input Options")
@@ -65,6 +65,7 @@ with st.sidebar:
     )
     progress_text = st.empty()
     progress_bar = st.progress(0)
+    download_placeholder = st.empty()  # This placeholder will hold the download button
 
 ###############################################################################
 # MAIN TITLE
@@ -72,7 +73,7 @@ with st.sidebar:
 st.title("Fire Detection: Original vs. Processed Video")
 
 ###############################################################################
-# Load YOLO model.
+# Load YOLO model
 ###############################################################################
 try:
     model = YOLO(model_path)
@@ -87,7 +88,7 @@ original_video_data = None
 processed_video_data = None  # For example pairs
 
 if example_option != "None":
-    # Load example videos from URLs.
+    # Use example videos from remote URLs.
     if example_option == "T Example":
         orig_url = "https://huggingface.co/spaces/tstone87/ccr-colorado/resolve/main/T1.mp4"
         proc_url = "https://huggingface.co/spaces/tstone87/ccr-colorado/resolve/main/T2.mpg"
@@ -100,17 +101,16 @@ if example_option != "None":
     except Exception as ex:
         st.error("Error loading example videos. Check your URLs.")
 else:
+    # No example selected. If a file is uploaded, use it.
     if source_file:
         file_type = source_file.type.split('/')[0]
         if file_type == 'image':
-            # For images, show the image preview.
             original_image = PIL.Image.open(source_file)
             buf = tempfile.NamedTemporaryFile(suffix=".png", delete=False)
             original_image.save(buf.name, format="PNG")
             with open(buf.name, "rb") as f:
                 original_video_data = f.read()
         else:
-            # For video uploads, save to a temporary file.
             tfile = tempfile.NamedTemporaryFile(delete=False, suffix=".mp4")
             tfile.write(source_file.read())
             tfile.flush()
@@ -133,27 +133,24 @@ with col1:
     else:
         st.info("No original video available.")
 
-# Create a placeholder for the processed video display.
-viewer_slot = col2.empty()
-col2.subheader("Result File")
-
-if example_option != "None":
-    if processed_video_data:
-        show_autoplay_video(processed_video_data, title="Processed Video")
-    else:
-        col2.info("No processed video available in example.")
-else:
-    if st.session_state["shortened_video_ready"] and st.session_state["shortened_video_data"]:
-        show_autoplay_video(st.session_state["shortened_video_data"], title="Processed Video")
+with col2:
+    st.subheader("Result File")
+    # Create a dedicated placeholder for the processed video.
+    viewer_slot = st.empty()
+    if example_option != "None":
+        if processed_video_data:
+            show_autoplay_video(processed_video_data, title="Processed Video")
+        else:
+            st.info("No processed video available in example.")
     else:
-        col2.info("Processed video will appear here once detection is run.")
+        viewer_slot.info("Processed video will appear here once detection is run.")
 
 ###############################################################################
 # DETECTION: Process the uploaded video if no example is selected.
 ###############################################################################
 if example_option == "None" and source_file and source_file.type.split('/')[0] != 'image':
     if st.sidebar.button("Let's Detect Wildfire"):
-        # Reset previous results.
+        # Reset previous processed results.
         st.session_state["processed_frames"] = []
         st.session_state["shortened_video_data"] = None
         st.session_state["shortened_video_ready"] = False
@@ -186,7 +183,6 @@ if example_option == "None" and source_file and source_file.type.split('/')[0] !
             sample_interval = 1
             output_fps = orig_fps
 
-        # Process frames.
         success, image = vidcap.read()
         while success:
             if frame_count % sample_interval == 0:
@@ -234,7 +230,7 @@ if example_option == "None" and source_file and source_file.type.split('/')[0] !
 # ALWAYS display the download button if a processed video is ready.
 ###############################################################################
 if st.session_state["shortened_video_ready"] and st.session_state["shortened_video_data"]:
-    st.download_button(
+    download_placeholder.download_button(
         label="Download Processed Video",
         data=st.session_state["shortened_video_data"],
         file_name="processed_video.mp4",