Fix properties update after changing creation/processing parameters interactively

Author: PierreRaybaut

datalab/gui/panel/base.py

@@ -374,27 +374,38 @@ def display_processing_history(self, obj: SignalObj | ImageObj) -> bool:
         self.processing_history.setText(history_text)
         return bool(history_text)
 
-    def update_properties_from(self, obj: SignalObj | ImageObj | None = None) -> None:
-        """Update properties from signal/image dataset
+    def __update_properties_dataset(self, obj: SignalObj | ImageObj) -> None:
+        """Update properties dataset from signal/image dataset
 
         Args:
             obj: Signal or Image object
         """
-        self.properties.setDisabled(obj is None)
-        if obj is None:
-            obj = self.objclass()
         dataset: SignalObj | ImageObj = self.properties.dataset
         dataset.set_defaults()
         update_dataset(dataset, obj)
         self.properties.get()
-        has_analysis_parameters = self.display_analysis_parameters(obj)
-        self.display_processing_history(obj)
         self.properties.apply_button.setEnabled(False)
 
+    def update_properties_from(self, obj: SignalObj | ImageObj | None = None) -> None:
+        """Update properties panel (properties, creation, processing) from object.
+
+        Args:
+            obj: Signal or Image object
+        """
+        self.properties.setDisabled(obj is None)
+        if obj is None:
+            obj = self.objclass()
+
+        # Update the properties dataset
+        self.__update_properties_dataset(obj)
         # Store original values to detect which properties have changed
-        # Using restore_dataset to convert the dataset to a dictionary
+        # (using `restore_dataset` to convert the dataset to a dictionary)
         self.__original_values = {}
-        restore_dataset(dataset, self.__original_values)
+        restore_dataset(self.properties.dataset, self.__original_values)
+
+        # Display analysis parameters and processing history
+        has_analysis_parameters = self.display_analysis_parameters(obj)
+        self.display_processing_history(obj)
 
         # Remove only Creation and Processing tabs (dynamic tabs)
         # Use widget references instead of text labels for reliable identification
@@ -595,6 +606,10 @@ def apply_creation_parameters(self) -> None:
         # of the Properties tab and could cause recursion issues)
         self.panel.refresh_plot(obj_uuid, update_items=True, force=True)
 
+        # Update the Properties tab to reflect the new object properties
+        # (e.g., data type, dimensions, etc.)
+        self.__update_properties_dataset(self.current_creation_obj)
+
         # Refresh the Creation tab with the new parameters
         # Use QTimer to defer this until after the current event is processed
         # Set the Creation tab as current to keep it visible after refresh
@@ -836,6 +851,10 @@ def apply_processing_parameters(
             self.panel.objview.update_item(obj_uuid)
             self.panel.refresh_plot(obj_uuid, update_items=True, force=True)
 
+            # Update the Properties tab to reflect the new object properties
+            # (e.g., data type, dimensions, etc.)
+            self.__update_properties_dataset(obj)
+
             # Refresh the Processing tab with the new parameters
             # Don't reset parameters from source object - keep the user's values
             # Set the Processing tab as current to keep it visible after refresh