Add multi-object property editing support

Allow applying property changes to multiple selected signals/images simultaneously.
Only modified properties are updated, preserving individual object settings.

Author: PierreRaybaut

CHANGELOG.md

@@ -6,6 +6,13 @@ See DataLab [roadmap page](https://datalab-platform.com/en/contributing/roadmap.
 
 💥 New features and enhancements:
 
+* Multi-object property editing:
+  * The properties panel now supports applying property changes to multiple selected objects simultaneously.
+  * When multiple signals or images are selected, modifying and applying properties updates all selected objects, not just the current one.
+  * Only the properties that were actually modified are applied to the selected objects, preserving individual object properties that were not changed.
+  * Typical use case: changing the LUT boundaries (Z scale bounds) or colormap of multiple images at once, or adjusting display properties for a group of signals.
+  * This significantly improves workflow efficiency when working with multiple objects that need similar property adjustments.
+
 * Internal console status indicator added to the status bar:
   * The status bar now features an indicator for the internal console, visible only when the console is hidden.
   * Clicking the indicator opens the internal console.

datalab/gui/panel/base.py

@@ -20,7 +20,7 @@
 import numpy as np
 import plotpy.io
 from guidata.configtools import get_icon
-from guidata.dataset import update_dataset
+from guidata.dataset import restore_dataset, update_dataset
 from guidata.qthelpers import add_actions, create_action, exec_dialog
 from plotpy.plot import PlotDialog
 from plotpy.tools import ActionTool
@@ -96,66 +96,137 @@ def is_plot_item_serializable(item: Any) -> bool:
 
 
 class ObjectProp(QW.QWidget):
-    """Object handling panel properties"""
+    """Object handling panel properties
 
-    def __init__(self, panel: BaseDataPanel, paramclass: SignalObj | ImageObj):
+    Args:
+        panel: parent data panel
+        objclass: class of the object handled by the panel (SignalObj or ImageObj)
+    """
+
+    def __init__(self, panel: BaseDataPanel, objclass: SignalObj | ImageObj) -> None:
         super().__init__(panel)
-        self.paramclass = paramclass
-        self.properties = gdq.DataSetEditGroupBox(_("Properties"), paramclass)
+        self.objclass = objclass
+        self.properties = gdq.DataSetEditGroupBox(_("Properties"), objclass)
         self.properties.SIG_APPLY_BUTTON_CLICKED.connect(panel.properties_changed)
         self.properties.setEnabled(False)
+        self.__original_values: dict[str, Any] = {}
 
         self.add_prop_layout = QW.QHBoxLayout()
         playout: QW.QGridLayout = self.properties.edit.layout
         playout.addLayout(
             self.add_prop_layout, playout.rowCount() - 1, 0, 1, 1, QC.Qt.AlignLeft
         )
 
-        self.param_label = QW.QLabel()
-        self.param_label.setTextFormat(QC.Qt.RichText)
-        self.param_label.setTextInteractionFlags(
+        self.analysis_parameter_label = QW.QLabel()
+        self.analysis_parameter_label.setTextFormat(QC.Qt.RichText)
+        self.analysis_parameter_label.setTextInteractionFlags(
             QC.Qt.TextBrowserInteraction | QC.Qt.TextSelectableByKeyboard
         )
-        self.param_label.setAlignment(QC.Qt.AlignTop)
-        param_scroll = QW.QScrollArea()
-        param_scroll.setWidgetResizable(True)
-        param_scroll.setWidget(self.param_label)
+        self.analysis_parameter_label.setAlignment(QC.Qt.AlignTop)
+        analysis_parameter_scroll = QW.QScrollArea()
+        analysis_parameter_scroll.setWidgetResizable(True)
+        analysis_parameter_scroll.setWidget(self.analysis_parameter_label)
 
         child: QW.QTabWidget = None
         for child in self.properties.children():
             if isinstance(child, QW.QTabWidget):
                 break
-        child.addTab(param_scroll, _("Analysis parameters"))
+        child.addTab(analysis_parameter_scroll, _("Analysis parameters"))
 
         vlayout = QW.QVBoxLayout()
         vlayout.addWidget(self.properties)
         self.setLayout(vlayout)
 
-    def add_button(self, button):
+    def add_button(self, button: QW.QPushButton) -> None:
         """Add additional button on bottom of properties panel"""
         self.add_prop_layout.addWidget(button)
 
-    def set_param_label(self, param: SignalObj | ImageObj):
-        """Set computing parameters label"""
+    def display_analysis_parameter(self, obj: SignalObj | ImageObj) -> None:
+        """Set analysis parameter label.
+
+        Args:
+            obj: Signal or Image object
+        """
         text = ""
-        for key, value in param.metadata.items():
+        for key, value in obj.metadata.items():
             if key.endswith("__param_html") and isinstance(value, str):
                 if text:
                     text += "<br><br>"
                 text += value
-        self.param_label.setText(text)
+        self.analysis_parameter_label.setText(text)
 
-    def update_properties_from(self, param: SignalObj | ImageObj | None = None):
-        """Update properties from signal/image dataset"""
-        self.properties.setDisabled(param is None)
-        if param is None:
-            param = self.paramclass()
-        self.properties.dataset.set_defaults()
-        update_dataset(self.properties.dataset, param)
+    def update_properties_from(self, obj: SignalObj | ImageObj | None = None) -> None:
+        """Update properties 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()
-        self.set_param_label(param)
+        self.display_analysis_parameter(obj)
         self.properties.apply_button.setEnabled(False)
 
+        # Store original values to detect which properties have changed
+        # Using restore_dataset to convert the dataset to a dictionary
+        self.__original_values = {}
+        restore_dataset(dataset, self.__original_values)
+
+    def get_changed_properties(self) -> dict[str, Any]:
+        """Get dictionary of properties that have changed from original values.
+
+        Returns:
+            Dictionary mapping property names to their new values, containing only
+            the properties that were modified by the user.
+        """
+        dataset = self.properties.dataset
+        changed = {}
+
+        # Get current values as a dictionary
+        current_values = {}
+        restore_dataset(dataset, current_values)
+
+        # Compare with original values
+        for key, current_value in current_values.items():
+            original_value = self.__original_values.get(key)
+            # Check if value has changed
+            if not self._values_equal(current_value, original_value):
+                changed[key] = current_value
+        return changed
+
+    def update_original_values(self) -> None:
+        """Update the stored original values to the current dataset values.
+
+        This should be called after applying changes to reset the baseline
+        for detecting future changes.
+        """
+        dataset = self.properties.dataset
+        self.__original_values = {}
+        restore_dataset(dataset, self.__original_values)
+
+    @staticmethod
+    def _values_equal(val1: Any, val2: Any) -> bool:
+        """Compare two values, handling special cases like numpy arrays.
+
+        Args:
+            val1: first value
+            val2: second value
+
+        Returns:
+            True if values are equal
+        """
+        # Handle numpy arrays
+        if isinstance(val1, np.ndarray) or isinstance(val2, np.ndarray):
+            if not isinstance(val1, np.ndarray) or not isinstance(val2, np.ndarray):
+                return False
+            return np.array_equal(val1, val2)
+        # Handle regular comparison
+        return val1 == val2
+
 
 class AbstractPanelMeta(type(QW.QSplitter), abc.ABCMeta):
     """Mixed metaclass to avoid conflicts"""
@@ -1381,13 +1452,21 @@ def selection_changed(self, update_items: bool = False) -> None:
     def properties_changed(self) -> None:
         """The properties 'Apply' button was clicked: update object properties,
         refresh plot and update object view."""
-        obj = self.objview.get_current_object()
-        # if obj is not None:  # XXX: Is it necessary?
-        obj.mark_roi_as_changed()
-        update_dataset(obj, self.objprop.properties.dataset)
-        self.objview.update_item(get_uuid(obj))
+        # Get only the properties that have changed from the original values
+        changed_props = self.objprop.get_changed_properties()
+
+        # Apply only the changed properties to all selected objects
+        for obj in self.objview.get_sel_objects(include_groups=True):
+            obj.mark_roi_as_changed()
+            # Update only the changed properties instead of all properties
+            update_dataset(obj, changed_props)
+            self.objview.update_item(get_uuid(obj))
         self.refresh_plot("selected", True, False)
 
+        # Update the stored original values to reflect the new state
+        # This ensures subsequent changes are compared against the current values
+        self.objprop.update_original_values()
+
     # ------Plotting data in modal dialogs----------------------------------------------
     def add_plot_items_to_dialog(self, dlg: PlotDialog, oids: list[str]) -> None:
         """Add plot items to dialog

doc/locale/fr/LC_MESSAGES/changelog.po

@@ -7,7 +7,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: DataLab \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-10-04 16:52+0200\n"
+"POT-Creation-Date: 2025-10-06 18:17+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language: fr\n"
@@ -30,6 +30,24 @@ msgstr "DataLab Version 1.0.0"
 msgid "💥 New features and enhancements:"
 msgstr "💥 Nouvelles fonctionnalités et améliorations :"
 
+msgid "Multi-object property editing:"
+msgstr "Édition des propriétés de plusieurs objets :"
+
+msgid "The properties panel now supports applying property changes to multiple selected objects simultaneously."
+msgstr "Le panneau des propriétés prend désormais en charge l'application des modifications de propriétés à plusieurs objets sélectionnés simultanément."
+
+msgid "When multiple signals or images are selected, modifying and applying properties updates all selected objects, not just the current one."
+msgstr "Lorsque plusieurs signaux ou images sont sélectionnés, la modification et l'application des propriétés mettent à jour tous les objets sélectionnés, et pas seulement le courant."
+
+msgid "Only the properties that were actually modified are applied to the selected objects, preserving individual object properties that were not changed."
+msgstr "Seules les propriétés qui ont été réellement modifiées sont appliquées aux objets sélectionnés, préservant les propriétés individuelles des objets qui n'ont pas été changées."
+
+msgid "Typical use case: changing the LUT boundaries (Z scale bounds) or colormap of multiple images at once, or adjusting display properties for a group of signals."
+msgstr "Cas d'utilisation typique : modification des limites de la LUT (bornes de l'échelle Z) ou de la carte des couleurs de plusieurs images à la fois, ou ajustement des propriétés d'affichage d'un groupe de signaux."
+
+msgid "This significantly improves workflow efficiency when working with multiple objects that need similar property adjustments."
+msgstr "Cela améliore considérablement l'efficacité du flux de travail lors de la manipulation de plusieurs objets nécessitant des ajustements de propriétés similaires."
+
 msgid "Internal console status indicator added to the status bar:"
 msgstr "Indicateur d'état de la console interne ajouté à la barre d'état :"
 

doc/locale/fr/LC_MESSAGES/features/general/remote.po

@@ -8,7 +8,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: DataLab \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-10-03 14:57+0200\n"
+"POT-Creation-Date: 2025-10-06 18:17+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <[email protected]>\n"
@@ -158,10 +158,10 @@ msgstr ""
 msgid "XML-RPC port to connect to. If not specified, the port is automatically retrieved from DataLab configuration."
 msgstr ""
 
-msgid "Timeout in seconds. Defaults to 5.0."
+msgid "Maximum time to wait for connection in seconds. Defaults to 5.0. This is the total maximum wait time, not per retry."
 msgstr ""
 
-msgid "Number of retries. Defaults to 10."
+msgid "Number of retries. Defaults to 10. This parameter is deprecated and will be removed in a future version (kept for backward compatibility)."
 msgstr ""
 
 msgid "Raises"
@@ -508,3 +508,4 @@ msgstr ""
 
 msgid "Show titles state"
 msgstr ""
+