update

app.py

@@ -1,6 +1,6 @@
 import gradio as gr
 from ui.dataset_input import create_dataset_input, load_example_dataset
-from ui.analysis_screen import create_analysis_screen  # Import create_analysis_screen instead 
+from ui.analysis_screen import create_analysis_screen, process_analysis_request
 
 def create_app():
     """
@@ -49,40 +49,49 @@ def create_app():
         
         # Analysis Tab
         with gr.Tab("Analysis"):
-            # Use create_analysis_screen instead of process_analysis_request
+            # Use create_analysis_screen to get UI components
             analysis_options, analysis_params, run_analysis_btn, analysis_output = create_analysis_screen()
-
+            
+            # Find the bow_top slider within analysis_params
+            # We directly access the bow_top slider using its name/element_id
+            bow_top_slider = None
+            for component in analysis_params.children:
+                if hasattr(component, 'children'):
+                    for child in component.children:
+                        if hasattr(child, 'elem_id') and child.elem_id == "bow_top_slider":
+                            bow_top_slider = child
+                            break
+            
             # Define a helper function to extract parameter values and call process_analysis_request
-            def run_analysis(dataset, selected_analyses, params_group):
-                from ui.analysis_screen import process_analysis_request
-                
+            def run_analysis(dataset, selected_analyses, bow_top_value=25):
                 # Check if dataset exists
                 if not dataset or "entries" not in dataset or not dataset["entries"]:
                     return {}, gr.update(visible=True, value={"error": "No dataset provided. Please create a dataset in the Dataset Input tab first."})
                 
-                # Extract actual parameter values from the Gradio components
-                # This is the key fix - we need to get the values from the sliders and inputs
-                params = {}
-                
-                # Get the bow_top value if Bag of Words is selected
-                if "Bag of Words" in selected_analyses:
-                    # Get the slider value - in a real app, you'd need to find a more robust way to do this
-                    # For now, we'll use the default value
-                    params["bow_top"] = 25
+                # Create parameters dictionary with the slider value
+                params = {"bow_top": bow_top_value}
                 
                 # Call the process_analysis_request function with proper parameters
                 results, output = process_analysis_request(dataset, selected_analyses, params)
                 return results, output
 
-            # Run analysis with proper parameters
-            run_analysis_btn.click(
-                fn=run_analysis,
-                inputs=[dataset_state, analysis_options, analysis_params],
-                outputs=[analysis_results_state, analysis_output]
-            )
+            # Run analysis with proper parameters - include the bow_top_slider directly
+            if bow_top_slider:
+                run_analysis_btn.click(
+                    fn=run_analysis,
+                    inputs=[dataset_state, analysis_options, bow_top_slider],
+                    outputs=[analysis_results_state, analysis_output]
+                )
+            else:
+                # Fallback if slider wasn't found
+                run_analysis_btn.click(
+                    fn=run_analysis,
+                    inputs=[dataset_state, analysis_options],
+                    outputs=[analysis_results_state, analysis_output]
+                )
     
     return app
 
 if __name__ == "__main__":
     app = create_app()
-    app.launch()
+    app.launch()