debug new

app.py

@@ -54,18 +54,22 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
     (script_input, tts_model_dropdown, pause_input,
      global_speed_input, global_instructions_input) = create_main_input_components(EFFECTIVE_MODEL_DEFAULT)
 
+    # Destructure the returned components from create_speaker_config_components
     (speaker_config_method_dropdown, single_voice_group, global_voice_dropdown,
      detailed_per_speaker_ui_group, load_per_speaker_ui_button,
-     dynamic_speaker_ui_area) = create_speaker_config_components()
+     dynamic_speaker_ui_wrapper) = create_speaker_config_components() # MODIFIED: use dynamic_speaker_ui_wrapper
 
     (calculate_cost_button, generate_button, cost_output,
      individual_lines_zip_output, merged_dialogue_mp3_output,
      status_output) = create_action_and_output_components()
 
+    # Event handler for TTS Model Dropdown
     tts_model_dropdown.change(
         fn=update_model_controls_visibility,
-        inputs=[tts_model_dropdown, script_input, speaker_configs_state, speaker_configs_state],
-        outputs=[global_speed_input, global_instructions_input, dynamic_speaker_ui_area, speaker_configs_state]
+        # CLEANUP: Removed duplicate speaker_configs_state
+        inputs=[tts_model_dropdown, script_input, speaker_configs_state], 
+        # MODIFIED: Output to the wrapper
+        outputs=[global_speed_input, global_instructions_input, dynamic_speaker_ui_wrapper, speaker_configs_state] 
     )
 
     speaker_config_method_dropdown.change(
@@ -74,11 +78,13 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
         outputs=[single_voice_group, detailed_per_speaker_ui_group]
     )
     
-    # MODIFIED: Button click now uses the new wrapper handle_load_refresh_button_click
+    # Event handler for the "Load/Refresh" Button
     load_per_speaker_ui_button.click(
-        fn=handle_load_refresh_button_click, # Use the new wrapper
-        inputs=[script_input, speaker_configs_state, tts_model_dropdown, speaker_configs_state], 
-        outputs=[dynamic_speaker_ui_area, speaker_configs_state]
+        fn=handle_load_refresh_button_click,
+        # CLEANUP: Removed duplicate speaker_configs_state
+        inputs=[script_input, speaker_configs_state, tts_model_dropdown], 
+        # MODIFIED: Output to the wrapper
+        outputs=[dynamic_speaker_ui_wrapper, speaker_configs_state] 
     )
     
     calculate_cost_button.click(

event_handlers.py

@@ -44,25 +44,16 @@ def handle_dynamic_input_change(new_value, current_configs_state_dict: dict, spe
     return current_configs_state_dict
 
 
-def load_refresh_per_speaker_ui_core(script_text: str, current_configs_state_dict: dict, tts_model: str, speaker_configs_state_component: gr.State):
+# CLEANUP: Modify function signatures to remove the redundant speaker_configs_state_comp argument
+def load_refresh_per_speaker_ui_core(script_text: str, current_configs_state_dict: dict, tts_model: str):
     """
     Core logic for generating per-speaker UI components.
-    Returns: (list_of_components_for_column, updated_state_dict)
+    Returns: (list_of_components_for_inner_column, updated_state_dict)
     """
-    # --- START OF PHASE 1 DEBUGGING (returns list of components directly) ---
-    print("DEBUG: load_refresh_per_speaker_ui_core CALLED - Phase 1: HARDCODED RETURN (direct list for Column)")
-    debug_markdown = gr.Markdown("## !! Dynamic Area Test Content (Button Click Path) !!")
-    return [debug_markdown], {} 
-    # --- END OF PHASE 1 DEBUGGING ---
-
-    # --- ORIGINAL LOGIC (Commented out for Phase 1) ---
-    # print(f"load_refresh_per_speaker_ui_core CALLED. TTS Model: {tts_model}") 
-    # unique_speakers = get_speakers_from_script(script_text)
-    # new_ui_components = []
-    # # ... (rest of original logic from previous load_refresh_per_speaker_ui) ...
-    # # Make sure this original logic path would also return:
-    # # return new_ui_components, current_configs_state_dict
-    # --- END OF ORIGINAL LOGIC ---
+    print(f"DEBUG: load_refresh_per_speaker_ui_core CALLED (for wrapper). TTS Model: {tts_model}")
+    # This list of components will be the children of the *new inner column*
+    debug_markdown = gr.Markdown("## !! Test Content for Inner Column (via Wrapper) !!")
+    return [debug_markdown], {} # Returns: (list_of_components, state_dictionary)
 
 
 async def handle_script_processing(
@@ -212,28 +203,41 @@ def handle_calculate_cost(dialogue_script: str, tts_model: str):
         return f"An unexpected error occurred during cost calculation: {str(e)}"
 
 # Wrapper for the "Load/Refresh Per-Speaker UI Button" click
-def handle_load_refresh_button_click(script_text: str, current_configs_state_dict: dict, tts_model: str, speaker_configs_state_comp: gr.State):
-    components_list, new_state_dict = load_refresh_per_speaker_ui_core(
-        script_text, current_configs_state_dict, tts_model, speaker_configs_state_comp
+# CLEANUP: Signature changed
+def handle_load_refresh_button_click(script_text: str, current_configs_state_dict: dict, tts_model: str):
+    components_list_for_inner_column, new_state_dict = load_refresh_per_speaker_ui_core(
+        script_text, current_configs_state_dict, tts_model # CLEANUP: Removed speaker_configs_state_comp
     )
-    # Return gr.update for the column, and the raw state dict for the gr.State component
-    return gr.update(children=components_list), new_state_dict
+    
+    # Create a new instance of the inner gr.Column with the dynamic children
+    new_inner_column = gr.Column(children=components_list_for_inner_column, elem_id="dynamic_ui_area_for_speakers")
+    
+    # Return gr.update for the wrapper group (to replace its children with the new_inner_column), 
+    # and the raw state dict for the gr.State component.
+    return gr.update(children=[new_inner_column]), new_state_dict
 
-def update_model_controls_visibility(selected_model: str, script_text_for_refresh: str, current_configs_state_dict: dict, speaker_configs_state_comp: gr.State):
+# Handler for TTS model dropdown change
+# CLEANUP: Signature changed
+def update_model_controls_visibility(selected_model: str, script_text_for_refresh: str, current_configs_state_dict: dict):
     """Updates visibility of global controls and refreshes per-speaker UI when TTS model changes."""
-    print(f"Model changed to: {selected_model}. Refreshing dynamic UI and controls.") 
+    print(f"Model changed to: {selected_model}. Refreshing dynamic UI (via wrapper) and controls.") 
     try:
-        # load_refresh_per_speaker_ui_core returns (list_of_components, updated_state_dict)
-        dynamic_ui_components_list, updated_state_dict = load_refresh_per_speaker_ui_core(
-            script_text_for_refresh, current_configs_state_dict, selected_model, speaker_configs_state_comp
+        dynamic_ui_children_list, updated_state_dict = load_refresh_per_speaker_ui_core(
+            script_text_for_refresh, current_configs_state_dict, selected_model # CLEANUP: Removed speaker_configs_state_comp
         )
-        # Wrap the list of components in gr.update(children=...) here
-        dynamic_ui_update_for_column = gr.update(children=dynamic_ui_components_list)
+        
+        # Create a new instance of the inner gr.Column
+        new_inner_column_for_model_change = gr.Column(children=dynamic_ui_children_list, elem_id="dynamic_ui_area_for_speakers")
+        
+        # This will be the update for the dynamic_speaker_ui_wrapper
+        dynamic_ui_update_for_wrapper = gr.update(children=[new_inner_column_for_model_change])
 
     except Exception as e:
         print(f"Error in load_refresh_per_speaker_ui_core called from model_controls_visibility: {e}")
         error_markdown = gr.Markdown(f"Error refreshing per-speaker UI: {e}")
-        dynamic_ui_update_for_column = gr.update(children=[error_markdown])
+        # Update the wrapper to show an error message inside a new inner column
+        error_inner_column = gr.Column(children=[error_markdown], elem_id="dynamic_ui_area_for_speakers")
+        dynamic_ui_update_for_wrapper = gr.update(children=[error_inner_column])
         updated_state_dict = current_configs_state_dict 
 
     is_tts1_family = selected_model in ["tts-1", "tts-1-hd"]
@@ -242,7 +246,7 @@ def update_model_controls_visibility(selected_model: str, script_text_for_refres
     return (
         gr.update(visible=is_tts1_family, interactive=is_tts1_family), 
         gr.update(visible=is_gpt_mini_tts, interactive=is_gpt_mini_tts), 
-        dynamic_ui_update_for_column, 
+        dynamic_ui_update_for_wrapper, # MODIFIED: This now updates the wrapper
         updated_state_dict      
     )
 

ui_layout.py

@@ -80,9 +80,24 @@ def create_speaker_config_components():
     with gr.Column(visible=(DEFAULT_SPEAKER_CONFIG_METHOD == "Detailed Configuration (Per Speaker UI)")) as detailed_per_speaker_ui_group:
         load_per_speaker_ui_button = gr.Button("Load/Refresh Per-Speaker Settings UI (from Script Above)")
         gr.Markdown("<small>Click button above to populate settings for each speaker found in the script. Settings are applied per-speaker. If script changes, click again to refresh.</small>")
-        dynamic_speaker_ui_area = gr.Column(elem_id="dynamic_ui_area_for_speakers")
+        
+        # NEW: Wrapper for the dynamic area
+        # This group will now be the target for updates.
+        dynamic_speaker_ui_wrapper = gr.Group(elem_id="dynamic_ui_wrapper_for_speakers")
+        with dynamic_speaker_ui_wrapper:
+             # The actual column is now a child of the wrapper.
+             # It can be initially empty or have a placeholder if desired.
+             dynamic_speaker_ui_area = gr.Column(elem_id="dynamic_ui_area_for_speakers") 
     
-    return speaker_config_method_dropdown, single_voice_group, global_voice_dropdown, detailed_per_speaker_ui_group, load_per_speaker_ui_button, dynamic_speaker_ui_area
+    # Ensure the new wrapper is returned correctly
+    return (
+        speaker_config_method_dropdown, 
+        single_voice_group, 
+        global_voice_dropdown, 
+        detailed_per_speaker_ui_group, 
+        load_per_speaker_ui_button, 
+        dynamic_speaker_ui_wrapper # MODIFIED: Return the wrapper
+    )
 
 def create_action_and_output_components():
     """Creates buttons for actions (cost, generate) and output display areas."""