Update app.py

app.py

@@ -1,4 +1,4 @@
-# app.py (The Final Polished Version)
+# app.py (Definitive Final Version with Syntax Fix)
 
 import gradio as gr
 from pathlib import Path
@@ -19,7 +19,7 @@ try:
 except FileNotFoundError:
     print("Warning: 'sample_images' directory not found."); SAMPLE_IMAGES = []
 
-# --- Core Logic Functions (Unchanged and Correct) ---
+# --- Core Logic Functions (Unchanged) ---
 async def process_analysis(patient_name, patient_age, image_list, is_sample=False):
     if not is_sample and (not patient_name or patient_age is None):
         raise gr.Error("Patient Name and Age are required.")
@@ -59,17 +59,17 @@ css = """
 /* --- Professional Dark Theme & Fonts --- */
 :root { --primary-hue: 220 !important; --secondary-hue: 210 !important; --neutral-hue: 210 !important; --body-background-fill: #111827 !important; --block-background-fill: #1F2937 !important; --block-border-width: 1px !important; --border-color-accent: #374151 !important; --background-fill-secondary: #1F2937 !important;}
 /* --- Header & Title Styling --- */
-#app_header { text-align: center; max-width: 900px; margin: 0 auto; } /* --- FIX: Center the header column --- */
+#app_header { text-align: center; max-width: 900px; margin: 0 auto; }
 #app_title { font-size: 2.8rem !important; font-weight: 700 !important; color: #FFFFFF !important; padding-top: 1rem; }
 #app_subtitle { font-size: 1.2rem !important; color: #9CA3AF !important; margin-bottom: 2rem; }
-/* --- Layout, Spacing, and Component Styling --- */
-#main_container { gap: 2rem; max-width: 900px; margin: 0 auto; } /* Center the main content */
+/* --- Layout and Spacing --- */
+#main_container { gap: 2rem; max-width: 900px; margin: 0 auto; }
 #results_gallery .gallery-item { padding: 0.25rem !important; background-color: #374151; border: 1px solid #374151 !important; }
 #bottom_controls { max-width: 500px; margin: 2.5rem auto 1rem auto; }
-#sample_gallery .gallery-item { border: 4px solid transparent; transition: border-color 0.3s ease; }
 """
 with gr.Blocks(theme=gr.themes.Default(primary_hue="blue", secondary_hue="blue"), css=css, title="Pneumonia Detection AI") as demo:
     
+    # --- UI Layout (Unchanged) ---
     with gr.Column() as main_app:
         with gr.Column(elem_id="app_header"):
             gr.Markdown("# 🩺 Pneumonia Detection AI", elem_id="app_title")
@@ -104,20 +104,10 @@ with gr.Blocks(theme=gr.themes.Default(primary_hue="blue", secondary_hue="blue")
             refresh_history_btn = gr.Button("Refresh History")
         history_df = gr.DataFrame(headers=["Name", "Age", "Prediction", "Confidence", "Date"], row_count=10, interactive=False)
 
-    # --- SAMPLES PAGE (THE DEFINITIVE FIX) ---
     with gr.Column(visible=False) as samples_page:
         gr.Markdown("# 🖼️ Sample Image Library", elem_classes="app_title")
         gr.Markdown("Click an image to run an anonymous analysis.")
-        
-        # We will use the gallery's native .select() event.
-        sample_gallery = gr.Gallery(
-            value=SAMPLE_IMAGES,
-            label="Sample Images",
-            columns=5, height=400,
-            allow_preview=True, # Allows the nice popup view
-            elem_id="sample_gallery"
-        )
-        # We add a hidden button that our code will "click"
+        sample_gallery = gr.Gallery(value=SAMPLE_IMAGES, label="Sample Images", columns=5, height=400, allow_preview=True, elem_id="sample_gallery")
         hidden_sample_analyze_btn = gr.Button("Analyze Sample", visible=False)
         back_to_main_btn_samp = gr.Button("⬅️ Back to Main App")
     
@@ -133,50 +123,38 @@ with gr.Blocks(theme=gr.themes.Default(primary_hue="blue", secondary_hue="blue")
     start_over_btn.click(fn=None, js="() => { window.location.reload(); }")
 
     # --- SAMPLE PAGE LOGIC (THE DEFINITIVE FIX) ---
-    
-    # When a sample image is clicked, this function runs.
-    # It takes the event data, which contains the path of the clicked image.
-    # Its ONLY job is to programmatically "click" the hidden analysis button.
     def on_sample_select(evt: gr.SelectData):
-        # We return the path of the selected image. This value will become the input
-        # for the hidden_sample_analyze_btn's click event.
         return evt.value
     
-    # The .select() event's output is now the INPUT to the hidden button's .click() event.
     sample_gallery.select(
         fn=on_sample_select,
-        None,
-        hidden_sample_analyze_btn
+        inputs=None, # The event data is passed automatically
+        outputs=[hidden_sample_analyze_btn]
     )
     
-    # The hidden button's click event runs the actual analysis
     async def handle_sample_analysis(selected_image_path: str):
-        if not selected_image_path: # This handles the case where nothing is selected
+        if not selected_image_path:
             raise gr.Error("Sample image path is missing.")
-            
         analysis_results = await process_analysis("Sample User", 0, [selected_image_path], is_sample=True)
-        
-        return [
-            gr.update(visible=True),   # main_app
-            gr.update(visible=False),  # samples_page
-            *analysis_results
-        ]
+        return [gr.update(visible=True), gr.update(visible=False), *analysis_results]
         
     hidden_sample_analyze_btn.click(
         fn=handle_sample_analysis, 
-        inputs=[hidden_sample_analyze_btn], # The button's value is the path
+        inputs=[hidden_sample_analyze_btn],
         outputs=[main_app, samples_page, uploader_column, results_column, result_images, result_label]
     )
     
-    # --- Page Navigation (Unchanged and Correct) ---
+    # --- Page Navigation ---
     all_pages = [main_app, history_page, samples_page]
     async def show_history_page_and_refresh(): records_update = await refresh_history_table(); return [gr.update(visible=False), gr.update(visible=True), gr.update(visible=False), records_update]
     def show_samples_page(): return [gr.update(visible=False), gr.update(visible=False), gr.update(visible=True)]
     def show_main_page(): return [gr.update(visible=True), gr.update(visible=False), gr.update(visible=False)]
+    
     history_btn.click(fn=show_history_page_and_refresh, outputs=all_pages + [history_df])
     samples_btn.click(fn=show_samples_page, outputs=all_pages)
     back_to_main_btn_hist.click(fn=show_main_page, outputs=all_pages)
     back_to_main_btn_samp.click(fn=show_main_page, outputs=all_pages)
+    
     refresh_history_btn.click(fn=refresh_history_table, outputs=history_df)
     demo.load(fn=refresh_history_table, outputs=history_df)