Enhance group short ID handling to distinguish between Signal and Image panels, improving clarity in cross-panel operations.

Author: PierreRaybaut

CHANGELOG.md

@@ -6,6 +6,12 @@ See DataLab [roadmap page](https://datalab-platform.com/en/contributing/roadmap.
 
 💥 New features and enhancements:
 
+* **Group short IDs now distinguish between panels**: Group identifiers are now panel-specific to avoid ambiguity in cross-panel operations
+  * Signal Panel groups use `gs` prefix (e.g., `gs001`, `gs002`, ...)
+  * Image Panel groups use `gi` prefix (e.g., `gi001`, `gi002`, ...)
+  * Cross-panel operation results now clearly indicate source panel (e.g., `radial_profile(gi001)` vs `radial_profile(gs001)`)
+  * Improves clarity when working with groups across both Signal and Image panels
+
 * **Performance optimizations for large result sets**: Added configurable display limits to prevent UI freezing with large analysis results
   * New configuration options to control result display behavior:
     * `max_shapes_to_draw` (default: 1,000): Maximum number of geometry shapes drawn on plots

datalab/gui/panel/base.py

@@ -1200,7 +1200,7 @@ def __init__(self, parent: QW.QWidget) -> None:
         self.mainwindow: DLMainWindow = parent
         self.objprop = ObjectProp(self, self.PARAMCLASS)
         self.show_label_checkbox: QW.QCheckBox | None = None
-        self.objmodel = objectmodel.ObjectModel()
+        self.objmodel = objectmodel.ObjectModel(f"g{self.PARAMCLASS.PREFIX}")
         self.objview = objectview.ObjectView(self, self.objmodel)
         self.objview.SIG_IMPORT_FILES.connect(self.handle_dropped_files)
         self.objview.populate_tree()

datalab/objectmodel.py

@@ -78,6 +78,8 @@ def set_uuid(obj: SignalObj | ImageObj | ObjectGroup) -> None:
 
 def get_short_id(obj: SignalObj | ImageObj | ObjectGroup) -> str:
     """Short object ID"""
+    if isinstance(obj, ObjectGroup):
+        return f"{obj.prefix}{get_number(obj):03d}"
     return f"{obj.PREFIX}{get_number(obj):03d}"
 
 
@@ -114,12 +116,12 @@ class ObjectGroup:
     Args:
         title: group title
         model: object model
+        prefix: prefix for short ID ("gs" for signal groups, "gi" for image groups)
     """
 
-    PREFIX = "g"
-
-    def __init__(self, title: str, model: ObjectModel) -> None:
+    def __init__(self, title: str, model: ObjectModel, prefix: str) -> None:
         self.model = model
+        self.prefix = prefix  # Instance-specific prefix
         self.uuid: str = str(uuid4())  # Group uuid
         self.number: int = 0  # Group number (used for short ID)
         self.__objects: list[str] = []  # list of object uuids
@@ -193,7 +195,13 @@ def get_object_ids(self) -> list[str]:
 class ObjectModel:
     """Represents a DataLab object model (groups of signals/images)"""
 
-    def __init__(self) -> None:
+    def __init__(self, group_prefix: str) -> None:
+        """Initialize object model
+
+        Args:
+            group_prefix: prefix for group short IDs ("gs" for signal, "gi" for image)
+        """
+        self._group_prefix = group_prefix
         # dict of objects, key is object uuid:
         self._objects: dict[str, SignalObj | ImageObj] = {}
         # list of groups:
@@ -360,8 +368,15 @@ def get_groups(self, uuids: list[str] | None = None) -> list[ObjectGroup]:
         return [group for group in self._groups if get_uuid(group) in uuids]
 
     def add_group(self, title: str) -> ObjectGroup:
-        """Add group to model"""
-        group = ObjectGroup(title, self)
+        """Add group to model
+
+        Args:
+            title: group title
+
+        Returns:
+            Created group object
+        """
+        group = ObjectGroup(title, self, self._group_prefix)
         self._groups.append(group)
         self.reset_short_ids()
         return group

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-11-09 18:55+0100\n"
+"POT-Creation-Date: 2025-11-09 19:35+0100\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language: fr\n"
@@ -30,6 +30,21 @@ msgstr "DataLab Version 1.0.0"
 msgid "💥 New features and enhancements:"
 msgstr "💥 Nouvelles fonctionnalités et améliorations :"
 
+msgid "**Group short IDs now distinguish between panels**: Group identifiers are now panel-specific to avoid ambiguity in cross-panel operations"
+msgstr "**Les identifiants courts de groupe distinguent désormais les panneaux** : Les identifiants de groupe sont désormais spécifiques au panneau afin d'éviter toute ambiguïté dans les opérations inter-panneaux"
+
+msgid "Signal Panel groups use `gs` prefix (e.g., `gs001`, `gs002`, ...)"
+msgstr "**Les groupes du panneau de signal utilisent le préfixe `gs` (par exemple, `gs001`, `gs002`, ...)**"
+
+msgid "Image Panel groups use `gi` prefix (e.g., `gi001`, `gi002`, ...)"
+msgstr "**Les groupes du panneau d'image utilisent le préfixe `gi` (par exemple, `gi001`, `gi002`, ...)**"
+
+msgid "Cross-panel operation results now clearly indicate source panel (e.g., `radial_profile(gi001)` vs `radial_profile(gs001)`)"
+msgstr "**Les résultats des opérations inter-panneaux indiquent désormais clairement le panneau source (par exemple, `radial_profile(gi001)` contre `radial_profile(gs001)`)**"
+
+msgid "Improves clarity when working with groups across both Signal and Image panels"
+msgstr "**Améliore la clarté lors du travail avec des groupes à travers les panneaux de signal et d'image**"
+
 msgid "**Performance optimizations for large result sets**: Added configurable display limits to prevent UI freezing with large analysis results"
 msgstr "**Optimisations des performances pour les grands ensembles de résultats** : Ajout de limites d'affichage configurables pour éviter le gel de l'interface utilisateur avec de grands résultats d'analyse"