Update app.py

app.py

@@ -1,5 +1,6 @@
 # app.py
 # (Showing relevant parts that need modification)
+
 import gradio as gr
 import pandas as pd
 import os
@@ -135,30 +136,34 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
             unique_ordered_sections = list(OrderedDict.fromkeys(pc["section"] for pc in plot_configs))
             num_unique_sections = len(unique_ordered_sections)
 
-            active_panel_action_state = gr.State(None)
-            explored_plot_id_state = gr.State(None)
+            active_panel_action_state = gr.State(None) # Stores {"plot_id": "...", "type": "insights/formula"}
+            explored_plot_id_state = gr.State(None) # Stores plot_id of the currently explored plot
 
             plot_ui_objects = {} 
             section_titles_map = {} 
 
             with gr.Row(equal_height=False):
                 with gr.Column(scale=8) as plots_area_col:
-                    ui_elements_tuple = build_analytics_tab_plot_area(plot_configs) # This function is in ui_generators.py
+                    # This function is in ui_generators.py and now builds the entire plot area
+                    ui_elements_tuple = build_analytics_tab_plot_area(plot_configs)
                     if isinstance(ui_elements_tuple, tuple) and len(ui_elements_tuple) == 2:
                         plot_ui_objects, section_titles_map = ui_elements_tuple
+                        # Validate section_titles_map
                         if not all(sec_name in section_titles_map for sec_name in unique_ordered_sections):
                             logging.error("section_titles_map from build_analytics_tab_plot_area is incomplete.")
+                            # Fallback for missing section titles
                             for sec_name in unique_ordered_sections:
                                 if sec_name not in section_titles_map:
+                                     # This should not happen if build_analytics_tab_plot_area is correct
                                      section_titles_map[sec_name] = gr.Markdown(f"### {sec_name} (Error Placeholder)") 
                     else:
-                        logging.error("build_analytics_tab_plot_area did not return (plot_ui_objects, section_titles_map).")
+                        logging.error("build_analytics_tab_plot_area did not return a tuple of (plot_ui_objects, section_titles_map).")
                         plot_ui_objects = ui_elements_tuple if isinstance(ui_elements_tuple, dict) else {} # Fallback
                         for sec_name in unique_ordered_sections: # Fallback for section titles
                             section_titles_map[sec_name] = gr.Markdown(f"### {sec_name} (Error Placeholder)")
 
 
-                with gr.Column(scale=4, visible=False) as global_actions_column_ui:
+                with gr.Column(scale=4, visible=False) as global_actions_column_ui: # Right-hand column for chatbot/formula
                     gr.Markdown("### 💡 Azioni Contestuali Grafico")
                     insights_chatbot_ui = gr.Chatbot(
                         label="Chat Insights", type="messages", height=450,
@@ -194,17 +199,20 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                 clicked_plot_config = next((p for p in plot_configs if p["id"] == plot_id_clicked), None)
                 if not clicked_plot_config:
                     logging.error(f"Config not found for plot_id {plot_id_clicked}")
-                    # Ensure this error return matches the expected number of outputs
                     num_plots = len(plot_configs)
-                    # Count for action_panel_outputs_list: 11 base UI + 4 states + N panels + 3N buttons + M sections = 15 + 4N + M
-                    # 15 + 4*19 + 6 = 15 + 76 + 6 = 97
-                    error_list_len = 11 + 4 + num_plots + (3 * num_plots) + num_unique_sections # Matches action_panel_outputs_list structure
+                    # 15 (base UI + states) + N (panels) + N (bomb_btns) + N (formula_btns) + N (explore_btns) + M (sections) = 15 + 4N + M
+                    error_list_len = 15 + (4 * num_plots) + num_unique_sections 
                     error_list = [gr.update()] * error_list_len
                     # Manually set key states to avoid further issues if possible
-                    error_list[11] = current_active_action_from_state # active_panel_action_state
-                    error_list[12] = current_chat_plot_id # current_chat_plot_id_st
-                    error_list[13] = current_chat_histories # chat_histories_st
-                    error_list[14] = current_explored_plot_id # explored_plot_id_state
+                    # Indices for states in action_panel_outputs_list:
+                    # active_panel_action_state is at index 11
+                    # current_chat_plot_id_st is at index 12
+                    # chat_histories_st is at index 13
+                    # explored_plot_id_state is at index 14
+                    error_list[11] = current_active_action_from_state 
+                    error_list[12] = current_chat_plot_id 
+                    error_list[13] = current_chat_histories 
+                    error_list[14] = current_explored_plot_id 
                     return error_list
 
                 clicked_plot_label = clicked_plot_config["label"]
@@ -212,6 +220,7 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                 hypothetical_new_active_state = {"plot_id": plot_id_clicked, "type": action_type}
                 is_toggling_off = current_active_action_from_state == hypothetical_new_active_state
 
+                # Initialize UI updates
                 action_col_visible_update = gr.update(visible=False)
                 insights_chatbot_visible_update, insights_chat_input_visible_update, insights_suggestions_row_visible_update = gr.update(visible=False), gr.update(visible=False), gr.update(visible=False)
                 formula_display_visible_update = gr.update(visible=False)
@@ -221,62 +230,81 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                 new_active_action_state_to_set, new_current_chat_plot_id = None, current_chat_plot_id
                 updated_chat_histories, new_explored_plot_id_to_set = current_chat_histories, current_explored_plot_id
 
-                panel_vis_updates, action_btn_updates, explore_btn_updates = [], [], []
+                # Lists for dynamic component updates
+                generated_panel_vis_updates = []
+                generated_bomb_btn_updates = []
+                generated_formula_btn_updates = []
+                generated_explore_btn_updates = []
                 section_title_vis_updates = [gr.update()] * num_unique_sections
 
+
                 if is_toggling_off:
-                    new_active_action_state_to_set = None
-                    action_col_visible_update = gr.update(visible=False) 
+                    new_active_action_state_to_set = None # Clear active state
+                    action_col_visible_update = gr.update(visible=False) # Hide the right action column
                     logging.info(f"Toggling OFF panel {action_type} for {plot_id_clicked}.")
+
+                    # Reset bomb and formula buttons to their default icons
+                    for _ in plot_configs:
+                        generated_bomb_btn_updates.append(gr.update(value=BOMB_ICON))
+                        generated_formula_btn_updates.append(gr.update(value=FORMULA_ICON))
+
                     if current_explored_plot_id: 
+                        # If an explore view was active, restore visibility based on that
                         explored_cfg = next((p for p in plot_configs if p["id"] == current_explored_plot_id), None)
                         explored_sec = explored_cfg["section"] if explored_cfg else None
                         for i, sec_name in enumerate(unique_ordered_sections):
                             section_title_vis_updates[i] = gr.update(visible=(sec_name == explored_sec))
                         for cfg in plot_configs:
                             is_exp = (cfg["id"] == current_explored_plot_id)
-                            panel_vis_updates.append(gr.update(visible=is_exp))
-                            explore_btn_updates.append(gr.update(value=ACTIVE_ICON if is_exp else EXPLORE_ICON))
+                            generated_panel_vis_updates.append(gr.update(visible=is_exp))
+                            generated_explore_btn_updates.append(gr.update(value=ACTIVE_ICON if is_exp else EXPLORE_ICON))
                     else: 
+                        # No explore view, so all plots and sections become visible
                         for i in range(num_unique_sections): section_title_vis_updates[i] = gr.update(visible=True)
                         for _ in plot_configs:
-                            panel_vis_updates.append(gr.update(visible=True))
-                            explore_btn_updates.append(gr.update(value=EXPLORE_ICON))
-                    for _ in plot_configs: 
-                        action_btn_updates.extend([gr.update(value=BOMB_ICON), gr.update(value=FORMULA_ICON)])
-                    if action_type == "insights": new_current_chat_plot_id = None
+                            generated_panel_vis_updates.append(gr.update(visible=True))
+                            generated_explore_btn_updates.append(gr.update(value=EXPLORE_ICON)) # Reset explore buttons
+                    
+                    if action_type == "insights": new_current_chat_plot_id = None # Clear chat context if insights panel is closed
 
-                else: 
+                else: # Toggling ON a new action or switching actions
                     new_active_action_state_to_set = hypothetical_new_active_state
-                    action_col_visible_update = gr.update(visible=True) 
-                    new_explored_plot_id_to_set = None 
-                    logging.info(f"Toggling ON panel {action_type} for {plot_id_clicked}. Cancelling explore view.")
+                    action_col_visible_update = gr.update(visible=True) # Show the right action column
+                    new_explored_plot_id_to_set = None # Cancel any active explore view
+                    logging.info(f"Toggling ON panel {action_type} for {plot_id_clicked}. Cancelling explore view if any.")
 
+                    # Set visibility for sections and plot panels: only the clicked plot and its section title
                     for i, sec_name in enumerate(unique_ordered_sections): 
                         section_title_vis_updates[i] = gr.update(visible=(sec_name == clicked_plot_section))
                     for cfg in plot_configs: 
-                        panel_vis_updates.append(gr.update(visible=(cfg["id"] == plot_id_clicked)))
-                        explore_btn_updates.append(gr.update(value=EXPLORE_ICON)) 
-                    
+                        generated_panel_vis_updates.append(gr.update(visible=(cfg["id"] == plot_id_clicked)))
+                        generated_explore_btn_updates.append(gr.update(value=EXPLORE_ICON)) # Reset all explore buttons
+
+                    # Update bomb and formula buttons: set active icon for the clicked one
                     for cfg_btn in plot_configs: 
                         is_act_ins = new_active_action_state_to_set == {"plot_id": cfg_btn["id"], "type": "insights"}
                         is_act_for = new_active_action_state_to_set == {"plot_id": cfg_btn["id"], "type": "formula"}
-                        action_btn_updates.extend([gr.update(value=ACTIVE_ICON if is_act_ins else BOMB_ICON), 
-                                                   gr.update(value=ACTIVE_ICON if is_act_for else FORMULA_ICON)])
+                        generated_bomb_btn_updates.append(gr.update(value=ACTIVE_ICON if is_act_ins else BOMB_ICON))
+                        generated_formula_btn_updates.append(gr.update(value=ACTIVE_ICON if is_act_for else FORMULA_ICON))
+                    
                     if action_type == "insights":
                         insights_chatbot_visible_update, insights_chat_input_visible_update, insights_suggestions_row_visible_update = gr.update(visible=True), gr.update(visible=True), gr.update(visible=True)
                         new_current_chat_plot_id = plot_id_clicked
                         history = current_chat_histories.get(plot_id_clicked, [])
                         summary = current_plot_data_for_chatbot.get(plot_id_clicked, f"No summary for '{clicked_plot_label}'.")
-                        if not history:
+                        if not history: # First time opening chat for this plot
                             prompt, sugg = get_initial_insight_prompt_and_suggestions(plot_id_clicked, clicked_plot_label, summary)
                             llm_hist = [{"role": "user", "content": prompt}]
+                            # Simulate LLM call for Gradio streaming if needed, or direct call
                             resp = await generate_llm_response(prompt, plot_id_clicked, clicked_plot_label, llm_hist, summary)
-                            history = [{"role": "assistant", "content": resp}]
+                            history = [{"role": "assistant", "content": resp}] # Start with AI's response
                             updated_chat_histories = {**current_chat_histories, plot_id_clicked: history}
-                        else: _, sugg = get_initial_insight_prompt_and_suggestions(plot_id_clicked, clicked_plot_label, summary)
+                        else: # Re-opening chat, just get suggestions
+                            _, sugg = get_initial_insight_prompt_and_suggestions(plot_id_clicked, clicked_plot_label, summary) # Get fresh suggestions
+                        
                         chatbot_content_update = gr.update(value=history)
                         s1_upd,s2_upd,s3_upd = gr.update(value=sugg[0] if sugg else "N/A"),gr.update(value=sugg[1] if len(sugg)>1 else "N/A"),gr.update(value=sugg[2] if len(sugg)>2 else "N/A")
+
                     elif action_type == "formula":
                         formula_display_visible_update = gr.update(visible=True)
                         formula_close_hint_visible_update = gr.update(visible=True) 
@@ -287,108 +315,155 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                             f_text += f"### {f_data['title']}\n\n{f_data['description']}\n\n**Calculation:**\n" + "\n".join([f"- {s}" for s in f_data['calculation_steps']])
                         else: f_text += "(No detailed formula information found.)"
                         formula_content_update = gr.update(value=f_text)
-                        new_current_chat_plot_id = None
+                        new_current_chat_plot_id = None # Clear chat context if formula panel is opened
 
+                # Assemble the full list of updates in the correct order
                 final_updates = [
                     action_col_visible_update, insights_chatbot_visible_update, chatbot_content_update,
                     insights_chat_input_visible_update, insights_suggestions_row_visible_update, 
                     s1_upd, s2_upd, s3_upd, formula_display_visible_update, formula_content_update,
-                    formula_close_hint_visible_update, 
+                    formula_close_hint_md, # This is the component for the hint's visibility
+                    # States
                     new_active_action_state_to_set, new_current_chat_plot_id, updated_chat_histories,
                     new_explored_plot_id_to_set 
                 ]
-                final_updates.extend(panel_vis_updates)
-                final_updates.extend(action_btn_updates) # Should be 2*N items
-                final_updates.extend(explore_btn_updates) # Should be N items
-                final_updates.extend(section_title_vis_updates)
-                # Total: 11 (base UI) + 4 (states) + N (panels) + 2N (action_btns) + N (explore_btns) + M (sections)
-                # = 15 + 4N + M. For N=19, M=6: 15 + 76 + 6 = 97. This matches action_panel_outputs_list.
+                final_updates.extend(generated_panel_vis_updates)    # N panel visibility updates
+                final_updates.extend(generated_bomb_btn_updates)     # N bomb button updates
+                final_updates.extend(generated_formula_btn_updates)  # N formula button updates
+                final_updates.extend(generated_explore_btn_updates)  # N explore button updates
+                final_updates.extend(section_title_vis_updates)      # M section title visibility updates
+                
+                # Expected length: 11 (base UI) + 4 (states) + N (panels) + N (bomb_btns) + N (formula_btns) + N (explore_btns) + M (sections)
+                # = 15 + 4N + M. For N=19, M=6: 15 + 4*19 + 6 = 15 + 76 + 6 = 97.
+                logging.debug(f"handle_panel_action returning {len(final_updates)} updates. Expected 97 for N=19, M=6.")
                 return final_updates
 
             async def handle_chat_message_submission(user_message: str, current_plot_id: str, chat_histories: dict, current_plot_data_for_chatbot: dict ):
                 if not current_plot_id or not user_message.strip():
-                    yield chat_histories.get(current_plot_id, []), gr.update(value=""), chat_histories; return
+                    # Ensure history is a list for the chatbot component
+                    current_history_for_plot = chat_histories.get(current_plot_id, [])
+                    if not isinstance(current_history_for_plot, list): current_history_for_plot = []
+                    yield current_history_for_plot, gr.update(value=""), chat_histories; return
+                
                 cfg = next((p for p in plot_configs if p["id"] == current_plot_id), None)
                 lbl = cfg["label"] if cfg else "Selected Plot"
                 summary = current_plot_data_for_chatbot.get(current_plot_id, f"No summary for '{lbl}'.")
-                hist = chat_histories.get(current_plot_id, []).copy() + [{"role": "user", "content": user_message}]
-                yield hist, gr.update(value=""), chat_histories
+                
+                # Ensure current history is a list
+                hist_for_plot = chat_histories.get(current_plot_id, [])
+                if not isinstance(hist_for_plot, list): hist_for_plot = [] # Safeguard
+
+                hist = hist_for_plot.copy() + [{"role": "user", "content": user_message}]
+                yield hist, gr.update(value=""), chat_histories # Show user message immediately
+
+                # Call LLM
                 resp = await generate_llm_response(user_message, current_plot_id, lbl, hist, summary)
                 hist.append({"role": "assistant", "content": resp})
-                yield hist, "", {**chat_histories, current_plot_id: hist}
+                
+                # Update chat histories state
+                updated_chat_histories = {**chat_histories, current_plot_id: hist}
+                yield hist, "", updated_chat_histories
+
 
             async def handle_suggested_question_click(suggestion_text: str, current_plot_id: str, chat_histories: dict, current_plot_data_for_chatbot: dict):
-                if not current_plot_id or not suggestion_text.strip():
-                    yield chat_histories.get(current_plot_id, []), gr.update(value=""), chat_histories; return
-                async for update in handle_chat_message_submission(suggestion_text, current_plot_id, chat_histories, current_plot_data_for_chatbot): yield update
+                if not current_plot_id or not suggestion_text.strip() or suggestion_text == "N/A":
+                    current_history_for_plot = chat_histories.get(current_plot_id, [])
+                    if not isinstance(current_history_for_plot, list): current_history_for_plot = []
+                    yield current_history_for_plot, gr.update(value=""), chat_histories; return
+                
+                # Use the generator to stream updates from handle_chat_message_submission
+                async for update_chunk in handle_chat_message_submission(suggestion_text, current_plot_id, chat_histories, current_plot_data_for_chatbot):
+                    yield update_chunk
+
 
             def handle_explore_click(plot_id_clicked, current_explored_plot_id_from_state, current_active_panel_action_state):
                 logging.info(f"Explore Click: Plot '{plot_id_clicked}'. Current Explored: {current_explored_plot_id_from_state}. Active Panel: {current_active_panel_action_state}")
                 num_plots = len(plot_configs)
                 if not plot_ui_objects: 
                     logging.error("plot_ui_objects not populated for handle_explore_click.")
-                    # explore_outputs_list: 4 base states/UI + N panels + N explore_btns + 2N action_btns + M sections
-                    # = 4 + 4N + M. For N=19, M=6: 4 + 76 + 6 = 86
-                    error_list_len = 4 + num_plots + num_plots + (2 * num_plots) + num_unique_sections
+                    # explore_outputs_list: 4 base states/UI + N panels + N explore_btns + N bomb_btns + N formula_btns + M sections
+                    error_list_len = 4 + (4 * num_plots) + num_unique_sections 
                     error_list = [gr.update()] * error_list_len
-                    error_list[0] = current_explored_plot_id_from_state
-                    error_list[2] = current_active_panel_action_state
+                    error_list[0] = current_explored_plot_id_from_state # explored_plot_id_state
+                    error_list[2] = current_active_panel_action_state # active_panel_action_state
                     return error_list
 
+                new_explored_id_to_set = None
+                is_toggling_off_explore = (plot_id_clicked == current_explored_plot_id_from_state)
 
-                new_explored_id_to_set, is_toggling_off_explore = None, (plot_id_clicked == current_explored_plot_id_from_state)
-                action_col_upd, new_active_panel_state_upd = gr.update(), current_active_panel_action_state
-                panel_vis, explore_btns, bomb_btns, formula_btns = [], [], [], []
-                section_title_vis = [gr.update()] * num_unique_sections
-                formula_hint_upd = gr.update(visible=False) 
+                action_col_upd = gr.update() # Default to no change for action column visibility
+                new_active_panel_state_upd = current_active_panel_action_state # Default to no change
+                formula_hint_upd = gr.update(visible=False) # Hide formula hint by default
+
+                panel_vis_updates = []
+                explore_btns_updates = []
+                bomb_btns_updates = []
+                formula_btns_updates = []
+                section_title_vis_updates = [gr.update()] * num_unique_sections
 
                 clicked_cfg = next((p for p in plot_configs if p["id"] == plot_id_clicked), None)
                 sec_of_clicked = clicked_cfg["section"] if clicked_cfg else None
 
                 if is_toggling_off_explore:
-                    new_explored_id_to_set = None
-                    logging.info(f"Stopping explore for {plot_id_clicked}. All plots/sections visible.")
-                    for i in range(num_unique_sections): section_title_vis[i] = gr.update(visible=True)
+                    new_explored_id_to_set = None # Clear explored state
+                    logging.info(f"Stopping explore for {plot_id_clicked}. All plots/sections to be visible.")
+                    for i in range(num_unique_sections): section_title_vis_updates[i] = gr.update(visible=True)
                     for _ in plot_configs:
-                        panel_vis.append(gr.update(visible=True)); explore_btns.append(gr.update(value=EXPLORE_ICON))
-                        bomb_btns.append(gr.update()); formula_btns.append(gr.update()) 
-                else: 
+                        panel_vis_updates.append(gr.update(visible=True))
+                        explore_btns_updates.append(gr.update(value=EXPLORE_ICON))
+                        # If an action panel was active, its buttons are handled by its own logic if it's closed.
+                        # Here, we just ensure explore buttons are reset. Bomb/Formula buttons remain as they were unless an active panel is closed.
+                        bomb_btns_updates.append(gr.update()) # No change unless active panel is closed
+                        formula_btns_updates.append(gr.update()) # No change
+                else: # Toggling ON explore for a plot
                     new_explored_id_to_set = plot_id_clicked
                     logging.info(f"Exploring {plot_id_clicked}. Hiding other plots/sections.")
-                    for i, sec_name in enumerate(unique_ordered_sections): section_title_vis[i] = gr.update(visible=(sec_name == sec_of_clicked))
-                    for cfg_idx, cfg in enumerate(plot_configs): # Iterate with index for bomb_btns/formula_btns
+                    for i, sec_name in enumerate(unique_ordered_sections): 
+                        section_title_vis_updates[i] = gr.update(visible=(sec_name == sec_of_clicked))
+                    for cfg in plot_configs:
                         is_target = (cfg["id"] == new_explored_id_to_set)
-                        panel_vis.append(gr.update(visible=is_target))
-                        explore_btns.append(gr.update(value=ACTIVE_ICON if is_target else EXPLORE_ICON))
-                        # Initialize bomb_btns and formula_btns correctly for all plots
-                        if current_active_panel_action_state and current_active_panel_action_state.get("plot_id") == cfg["id"]:
-                             # This should not happen if we are closing active panel below
-                            pass # Will be overridden
-                        
-                    if current_active_panel_action_state: 
+                        panel_vis_updates.append(gr.update(visible=is_target))
+                        explore_btns_updates.append(gr.update(value=ACTIVE_ICON if is_target else EXPLORE_ICON))
+                    
+                    # If an action panel (insights/formula) is currently active, close it
+                    if current_active_panel_action_state:
                         logging.info("Closing active insight/formula panel due to explore click.")
-                        action_col_upd = gr.update(visible=False); new_active_panel_state_upd = None
-                        # Reset all bomb and formula buttons
-                        bomb_btns = [gr.update(value=BOMB_ICON) for _ in plot_configs]
-                        formula_btns = [gr.update(value=FORMULA_ICON) for _ in plot_configs]
-                    else: 
-                        # If no panel was active, keep current state of bomb/formula buttons (no update)
-                        bomb_btns = [gr.update() for _ in plot_configs]
-                        formula_btns = [gr.update() for _ in plot_configs]
+                        action_col_upd = gr.update(visible=False) # Hide the right action column
+                        new_active_panel_state_upd = None # Clear the active panel state
+                        formula_hint_upd = gr.update(visible=False) # Hide formula hint
+                        # Reset all bomb and formula buttons to default icons
+                        for _ in plot_configs:
+                            bomb_btns_updates.append(gr.update(value=BOMB_ICON))
+                            formula_btns_updates.append(gr.update(value=FORMULA_ICON))
+                    else:
+                        # No action panel was active, bomb/formula buttons remain as they are (no update)
+                        for _ in plot_configs:
+                            bomb_btns_updates.append(gr.update())
+                            formula_btns_updates.append(gr.update())
+                
+                # Order: states/UI (4), panels (N), explore_btns (N), bomb_btns (N), formula_btns (N), section_titles (M)
+                # Total: 4 + 4N + M.
+                final_explore_updates = [
+                    new_explored_id_to_set, action_col_upd, new_active_panel_state_upd, formula_hint_upd
+                ]
+                final_explore_updates.extend(panel_vis_updates)
+                final_explore_updates.extend(explore_btns_updates)
+                final_explore_updates.extend(bomb_btns_updates)
+                final_explore_updates.extend(formula_btns_updates)
+                final_explore_updates.extend(section_title_vis_updates)
                 
-                # Order: states (4), panels (N), explore_btns (N), bomb_btns (N), formula_btns (N), section_titles (M)
-                # Total: 4 + N + N + N + N + M = 4 + 4N + M.
-                return [new_explored_id_to_set, action_col_upd, new_active_panel_state_upd, formula_hint_upd] + \
-                       panel_vis + explore_btns + bomb_btns + formula_btns + section_title_vis
+                logging.debug(f"handle_explore_click returning {len(final_explore_updates)} updates. Expected {4 + 4*len(plot_configs) + num_unique_sections}.")
+                return final_explore_updates
+
 
             # --- Define Output Lists for Event Handlers ---
             # Base UI for action panel (insights/formula): 11 elements
             _base_action_panel_ui_outputs = [
-                global_actions_column_ui, insights_chatbot_ui, insights_chatbot_ui, 
+                global_actions_column_ui, insights_chatbot_ui, insights_chatbot_ui, # Chatbot vis + value
                 insights_chat_input_ui, insights_suggestions_row_ui, 
                 insights_suggestion_1_btn, insights_suggestion_2_btn, insights_suggestion_3_btn,
                 formula_display_markdown_ui, formula_display_markdown_ui, # Formula MD (vis + value)
-                formula_close_hint_md 
+                formula_close_hint_md # Formula hint visibility
             ]
             # States for action panel: 4 elements
             _action_panel_state_outputs = [active_panel_action_state, current_chat_plot_id_st, chat_histories_st, explored_plot_id_state]
@@ -399,7 +474,7 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
             action_panel_outputs_list.extend([plot_ui_objects.get(pc["id"], {}).get("formula_button", gr.update()) for pc in plot_configs]) # +N (formula_button)
             action_panel_outputs_list.extend([plot_ui_objects.get(pc["id"], {}).get("explore_button", gr.update()) for pc in plot_configs]) # +N (explore_button)
             action_panel_outputs_list.extend([section_titles_map.get(s_name, gr.update()) for s_name in unique_ordered_sections]) # +M (sections)
-            # Total for action_panel_outputs_list = 15 + 4N + M. For N=19, M=6: 15 + 76 + 6 = 97.
+            # Total for action_panel_outputs_list = 15 + 4N + M. For N=19, M=6: 15 + 76 + 6 = 97. Correct.
 
             # Base UI/States for explore click: 4 elements
             _explore_base_outputs = [explored_plot_id_state, global_actions_column_ui, active_panel_action_state, formula_close_hint_md] 
@@ -409,7 +484,7 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
             explore_outputs_list.extend([plot_ui_objects.get(pc["id"], {}).get("bomb_button", gr.update()) for pc in plot_configs])    # +N (bomb_button)
             explore_outputs_list.extend([plot_ui_objects.get(pc["id"], {}).get("formula_button", gr.update()) for pc in plot_configs]) # +N (formula_button)
             explore_outputs_list.extend([section_titles_map.get(s_name, gr.update()) for s_name in unique_ordered_sections]) # +M (sections)
-            # Total for explore_outputs_list = 4 + 4N + M. For N=19, M=6: 4 + 76 + 6 = 86.
+            # Total for explore_outputs_list = 4 + 4N + M. For N=19, M=6: 4 + 76 + 6 = 86. Correct.
 
             action_click_inputs = [active_panel_action_state, chat_histories_st, current_chat_plot_id_st, plot_data_for_chatbot_st, explored_plot_id_state]
             explore_click_inputs = [explored_plot_id_state, active_panel_action_state]
@@ -423,52 +498,76 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                 plot_id = config_item["id"]
                 if plot_id in plot_ui_objects:
                     ui_obj = plot_ui_objects[plot_id]
-                    ui_obj["bomb_button"].click(fn=create_panel_action_handler(plot_id, "insights"), inputs=action_click_inputs, outputs=action_panel_outputs_list, api_name=f"action_insights_{plot_id}")
-                    ui_obj["formula_button"].click(fn=create_panel_action_handler(plot_id, "formula"), inputs=action_click_inputs, outputs=action_panel_outputs_list, api_name=f"action_formula_{plot_id}")
-                    ui_obj["explore_button"].click(fn=lambda current_explored_val, current_active_panel_val, p_id=plot_id: handle_explore_click(p_id, current_explored_val, current_active_panel_val), inputs=explore_click_inputs, outputs=explore_outputs_list, api_name=f"action_explore_{plot_id}")
+                    # Ensure buttons exist before attaching handlers
+                    if ui_obj.get("bomb_button"):
+                        ui_obj["bomb_button"].click(fn=create_panel_action_handler(plot_id, "insights"), inputs=action_click_inputs, outputs=action_panel_outputs_list, api_name=f"action_insights_{plot_id}")
+                    if ui_obj.get("formula_button"):
+                        ui_obj["formula_button"].click(fn=create_panel_action_handler(plot_id, "formula"), inputs=action_click_inputs, outputs=action_panel_outputs_list, api_name=f"action_formula_{plot_id}")
+                    if ui_obj.get("explore_button"):
+                        ui_obj["explore_button"].click(fn=lambda current_explored_val, current_active_panel_val, p_id=plot_id: handle_explore_click(p_id, current_explored_val, current_active_panel_val), inputs=explore_click_inputs, outputs=explore_outputs_list, api_name=f"action_explore_{plot_id}")
                 else: logging.warning(f"UI object for plot_id '{plot_id}' not found for click handlers.")
 
+
             chat_submission_outputs = [insights_chatbot_ui, insights_chat_input_ui, chat_histories_st]
             chat_submission_inputs = [insights_chat_input_ui, current_chat_plot_id_st, chat_histories_st, plot_data_for_chatbot_st]
             insights_chat_input_ui.submit(fn=handle_chat_message_submission, inputs=chat_submission_inputs, outputs=chat_submission_outputs, api_name="submit_chat_message")
-            suggestion_click_inputs = [current_chat_plot_id_st, chat_histories_st, plot_data_for_chatbot_st]
-            insights_suggestion_1_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_1_btn] + suggestion_click_inputs, outputs=chat_submission_outputs, api_name="click_suggestion_1")
-            insights_suggestion_2_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_2_btn] + suggestion_click_inputs, outputs=chat_submission_outputs, api_name="click_suggestion_2")
-            insights_suggestion_3_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_3_btn] + suggestion_click_inputs, outputs=chat_submission_outputs, api_name="click_suggestion_3")
+            
+            suggestion_click_inputs_base = [current_chat_plot_id_st, chat_histories_st, plot_data_for_chatbot_st]
+            insights_suggestion_1_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_1_btn] + suggestion_click_inputs_base, outputs=chat_submission_outputs, api_name="click_suggestion_1")
+            insights_suggestion_2_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_2_btn] + suggestion_click_inputs_base, outputs=chat_submission_outputs, api_name="click_suggestion_2")
+            insights_suggestion_3_btn.click(fn=handle_suggested_question_click, inputs=[insights_suggestion_3_btn] + suggestion_click_inputs_base, outputs=chat_submission_outputs, api_name="click_suggestion_3")
+
 
             def refresh_all_analytics_ui_elements(current_token_state, date_filter_val, custom_start_val, custom_end_val, current_chat_histories):
                 logging.info("Refreshing all analytics UI elements and resetting actions/chat.")
                 plot_gen_results = update_analytics_plots_figures(current_token_state, date_filter_val, custom_start_val, custom_end_val, plot_configs)
                 status_msg, gen_figs, new_summaries = plot_gen_results[0], plot_gen_results[1:-1], plot_gen_results[-1]
                 
-                all_updates = [status_msg] # 1
-                all_updates.extend(gen_figs if len(gen_figs) == len(plot_configs) else [create_placeholder_plot("Error", f"Fig missing {i}") for i in range(len(plot_configs))]) # +N (19)
+                all_updates = [status_msg] # 1. analytics_status_md
+                all_updates.extend(gen_figs if len(gen_figs) == len(plot_configs) else [create_placeholder_plot("Error", f"Fig missing {i}") for i in range(len(plot_configs))]) # +N (plot figures)
                 
-                # UI Resets (9 elements) + State Resets (4 elements) = 13 elements
+                # UI Resets for action panel (9 elements)
                 all_updates.extend([ 
-                    gr.update(visible=False), gr.update(value=[], visible=False), gr.update(value="", visible=False), # global_actions_col, chatbot, chat_input
-                    gr.update(visible=False), gr.update(value="S1"), gr.update(value="S2"), gr.update(value="S3"), # suggestions_row, sugg_btns
-                    gr.update(value="Formula details here.", visible=False), gr.update(visible=False), # formula_md, formula_hint_md
-                    None, None, {}, new_summaries # active_panel_state, current_chat_plot_id, chat_histories, plot_data_for_chatbot
-                ]) # +13
+                    gr.update(visible=False), # global_actions_column_ui
+                    gr.update(value=[], visible=False), # insights_chatbot_ui (value + visibility)
+                    gr.update(value="", visible=False), # insights_chat_input_ui
+                    gr.update(visible=False), # insights_suggestions_row_ui
+                    gr.update(value="S1"), gr.update(value="S2"), gr.update(value="S3"), # sugg_btns
+                    gr.update(value="Formula details here.", visible=False), # formula_display_markdown_ui (value + visibility)
+                    gr.update(visible=False) # formula_close_hint_md
+                ]) 
                 
-                for _ in plot_configs: # Reset button icons (3) & panel visibility (1) = 4 per plot
-                    all_updates.extend([gr.update(value=BOMB_ICON), gr.update(value=FORMULA_ICON), gr.update(value=EXPLORE_ICON), gr.update(visible=True)]) # +4N (76)
+                # State Resets (4 elements)
+                all_updates.extend([ 
+                    None, # active_panel_action_state
+                    None, # current_chat_plot_id_st
+                    {},   # chat_histories_st (reset all chat histories)
+                    new_summaries # plot_data_for_chatbot_st
+                ]) 
+                
+                # Plot-specific buttons (3 per plot: bomb, formula, explore) and panel visibility (1 per plot)
+                for _ in plot_configs: 
+                    all_updates.extend([
+                        gr.update(value=BOMB_ICON), 
+                        gr.update(value=FORMULA_ICON), 
+                        gr.update(value=EXPLORE_ICON), 
+                        gr.update(visible=True) # panel_component visibility
+                    ]) 
                 
                 all_updates.append(None) # Reset explored_plot_id_state (1 element)
-                all_updates.extend([gr.update(visible=True)] * num_unique_sections) # Make all section titles visible (M = 6 elements)
-                # Total: 1 + N + 13 + 4N + 1 + M = 1 + 19 + 13 + 76 + 1 + 6 = 116. This matches apply_filter_and_sync_outputs_list.
-                logging.info(f"Prepared {len(all_updates)} updates for analytics refresh.")
+                all_updates.extend([gr.update(visible=True)] * num_unique_sections) # Make all section titles visible (M elements)
+                
+                # Total: 1 (status) + N (figs) + 9 (UI resets) + 4 (state resets) + 4N (plot buttons/panels) + 1 (explored_state) + M (sections)
+                # = 1 + N + 9 + 4 + 4N + 1 + M = 15 + 5N + M.
+                # For N=19, M=6: 15 + 5*19 + 6 = 15 + 95 + 6 = 116. This matches apply_filter_and_sync_outputs_list.
+                logging.info(f"Prepared {len(all_updates)} updates for analytics refresh. Expected {15 + 5*len(plot_configs) + num_unique_sections}.")
                 return all_updates
 
             # Construction of apply_filter_and_sync_outputs_list
-            # This list defines the components Gradio expects to be updated.
-            # Its length must match the number of items returned by refresh_all_analytics_ui_elements.
-            
             apply_filter_and_sync_outputs_list = [analytics_status_md] # 1. Status
             apply_filter_and_sync_outputs_list.extend([plot_ui_objects.get(pc["id"], {}).get("plot_component", gr.update()) for pc in plot_configs]) # 2. Plot figures (N)
             
-            # 3. UI Resets (9 elements for action panel)
+            # 3. UI Resets for action panel (9 elements)
             _ui_resets_for_filter = [
                 global_actions_column_ui, insights_chatbot_ui, insights_chat_input_ui,
                 insights_suggestions_row_ui, insights_suggestion_1_btn, insights_suggestion_2_btn, insights_suggestion_3_btn,
@@ -487,7 +586,7 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
                     plot_ui_objects.get(pid, {}).get("bomb_button", gr.update()),
                     plot_ui_objects.get(pid, {}).get("formula_button", gr.update()),
                     plot_ui_objects.get(pid, {}).get("explore_button", gr.update()),
-                    plot_ui_objects.get(pid, {}).get("panel_component", gr.update())
+                    plot_ui_objects.get(pid, {}).get("panel_component", gr.update()) # Panel visibility
                 ])
             
             # 6. Explored state reset (1 element)
@@ -495,8 +594,7 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
             
             # 7. Section Titles (M elements)
             apply_filter_and_sync_outputs_list.extend([section_titles_map.get(s_name, gr.update()) for s_name in unique_ordered_sections]) 
-            # Total elements: 1 + N + 9 + 4 + 4N + 1 + M = 15 + 5N + M
-            # For N=19, M=6: 15 + 5*19 + 6 = 15 + 95 + 6 = 110 + 6 = 116. This is correct.
+            # Total elements: 1 + N + 9 + 4 + 4N + 1 + M = 15 + 5N + M. Correct.
             
             apply_filter_btn.click(
                 fn=refresh_all_analytics_ui_elements,
@@ -534,10 +632,11 @@ with gr.Blocks(theme=gr.themes.Soft(primary_hue="blue", secondary_hue="sky"),
     sync_event_part3 = sync_event_part2.then(fn=display_main_dashboard, inputs=[token_state], outputs=[dashboard_display_html], show_progress=False)
     sync_event_final = sync_event_part3.then(fn=refresh_all_analytics_ui_elements, inputs=[token_state, date_filter_selector, custom_start_date_picker, custom_end_date_picker, chat_histories_st], outputs=apply_filter_and_sync_outputs_list, show_progress="full")
 
+
 if __name__ == "__main__":
     if not os.environ.get(LINKEDIN_CLIENT_ID_ENV_VAR): logging.warning(f"ATTENZIONE: '{LINKEDIN_CLIENT_ID_ENV_VAR}' non impostata.")
     if not all(os.environ.get(var) for var in [BUBBLE_APP_NAME_ENV_VAR, BUBBLE_API_KEY_PRIVATE_ENV_VAR, BUBBLE_API_ENDPOINT_ENV_VAR]):
         logging.warning("ATTENZIONE: Variabili Bubble non impostate.")
     try: logging.info(f"Matplotlib: {matplotlib.__version__}, Backend: {matplotlib.get_backend()}")
     except ImportError: logging.warning("Matplotlib non trovato.")
-    app.launch(server_name="0.0.0.0", server_port=7860, debug=True)
+    app.launch(server_name="0.0.0.0", server_port=7860, debug=True)