fixed problem with updating

Author: PaulJonasJost

src/petab_gui/views/main_view.py

@@ -126,16 +126,16 @@ def __init__(self):
 
         self.tab_widget.currentChanged.connect(self.set_docks_visible)
 
+        # Track if we're in a minimize/restore cycle (must be set before load_ui_settings)
+        self._was_minimized = False
+
         settings_manager.load_ui_settings(self)
 
         # drag drop
         self.setAcceptDrops(True)
 
         self.find_replace_bar = None
 
-        # Track if we're in a minimize/restore cycle
-        self._was_minimized = False
-
     def default_view(self):
         """Reset the view to a fixed 3x2 grid using manual geometry."""
         if hasattr(self, "dock_visibility"):

src/petab_gui/views/simple_plot_view.py

@@ -105,15 +105,14 @@ def _get_cached_df(self, table_name, proxy_model):
 
     def _connect_proxy_signals(self, proxy, cache_key):
         """Connect proxy signals for cache invalidation and plotting."""
-        for signal in [
-            proxy.dataChanged,
-            proxy.rowsInserted,
-            proxy.rowsRemoved,
-        ]:
-            signal.connect(
-                lambda *, key=cache_key: self._invalidate_cache(key)
-            )
-            signal.connect(self._debounced_plot)
+
+        def on_data_change(*args, **kwargs):
+            self._invalidate_cache(cache_key)
+            self._debounced_plot()
+
+        proxy.dataChanged.connect(on_data_change)
+        proxy.rowsInserted.connect(on_data_change)
+        proxy.rowsRemoved.connect(on_data_change)
 
     def initialize(
         self, meas_proxy, sim_proxy, cond_proxy, vis_proxy, petab_model
@@ -124,6 +123,10 @@ def initialize(
         self.vis_proxy = vis_proxy
         self.petab_model = petab_model
 
+        # Clear all cache when reinitializing
+        for key in self._cache_valid:
+            self._cache_valid[key] = False
+
         # Connect cache invalidation and data changes
         self.options_manager.option_changed.connect(self._debounced_plot)
 
@@ -218,6 +221,9 @@ def _render_on_main_thread(self, payload):
         self._update_tabs(fig)
 
     def _update_tabs(self, fig: plt.Figure):
+        # Save current tab index before clearing
+        current_tab_index = self.tab_widget.currentIndex()
+
         # Clean previous tabs
         self.tab_widget.clear()
         # Clear Highlighter
@@ -329,6 +335,10 @@ def _update_tabs(self, fig: plt.Figure):
         # Plot residuals if necessary
         self.plot_residuals()
 
+        # Restore the previously selected tab (if valid)
+        if 0 <= current_tab_index < self.tab_widget.count():
+            self.tab_widget.setCurrentIndex(current_tab_index)
+
     def highlight_from_selection(
         self, selected_rows: list[int], proxy=None, y_axis_col="measurement"
     ):
@@ -556,8 +566,13 @@ def __init__(self, canvas, parent):
         self.addWidget(self.settings_btn)
 
     def update_checked_state(self, selected_option):
-        for action in self.groupy_by_options.values():
-            action.setChecked(action.text() == f"Groupy by {selected_option}")
+        for grp, action in self.groupy_by_options.items():
+            if grp == "vis_df":
+                action.setChecked(selected_option == "vis_df")
+            else:
+                action.setChecked(
+                    action.text() == f"Group by {selected_option}"
+                )
 
 
 def create_plot_tab(