Plot results: Systematic Results group with source IDs in titles

- Always use "Results" group for all plot results operations
- Include source object IDs in result signal titles
- Use compact format (first 2, ..., last) for many objects
- Update tests for systematic behavior

Author: PierreRaybaut

CHANGELOG.md

@@ -52,6 +52,18 @@ See DataLab [roadmap page](https://datalab-platform.com/en/contributing/roadmap.
   * Clipboard-based workflow enables efficient annotation reuse across objects
   * Independent from ROI management - annotations and ROIs can coexist on same object
 
+* **Plot results improvements**: Enhanced result plotting with better organization and informative titles
+  * **Results group creation**: All plot results operations now automatically organize results in a dedicated "Results" group
+    * First time plotting results creates a new "Results" group in the signal panel
+    * Subsequent result plots reuse the same "Results" group for better organization
+    * Keeps workspace clean by consolidating all results in one well-defined location
+    * Consistent behavior regardless of whether groups or individual objects are selected
+  * **Comprehensive result titles**: Result signal titles now include source object identifiers
+    * For "one curve per title" mode: titles include all source object short IDs (e.g., "FWHM (s001, s002, s003): Δx = f(indices)")
+    * For many source objects (>3): uses compact format showing first 2 IDs, "...", then last ID (e.g., "FWHM (s001, s002, ..., s010): Δx = f(indices)")
+    * Makes it easy to identify which objects contributed to plotted results
+    * Maintains consistency with "one curve per object" mode which already included IDs
+
 * **Metadata management improvements**: Enhanced metadata handling with clipboard-based workflow
   * Reorganized Edit menu with new "Metadata" submenu grouping all metadata operations
   * **Paste metadata** action now only enabled when metadata clipboard is not empty

datalab/gui/panel/base.py

@@ -2813,8 +2813,19 @@ def __add_result_signal(
         title: str,
         xaxis: str,
         yaxis: str,
+        group_id: str | None = None,
     ) -> None:
-        """Add result signal"""
+        """Add result signal
+
+        Args:
+            x: X data
+            y: Y data
+            title: Signal title
+            xaxis: X axis label
+            yaxis: Y axis label
+            group_id: UUID of the group to add the signal to. If None, add to
+             current group.
+        """
         xdata = np.array(x, dtype=float)
         ydata = np.array(y, dtype=float)
 
@@ -2824,7 +2835,7 @@ def __add_result_signal(
             y=ydata,
             labels=[xaxis, yaxis],
         )
-        self.mainwindow.signalpanel.add_object(obj)
+        self.mainwindow.signalpanel.add_object(obj, group_id=group_id)
 
     def __create_plot_result_param_class(self, rdata: ResultData) -> type[gds.DataSet]:
         """Create PlotResultParam class dynamically based on result data.
@@ -2878,6 +2889,7 @@ def __plot_result(
         rdata: ResultData,
         objs: list[SignalObj | ImageObj],
         param: gds.DataSet | None = None,
+        result_group_id: str | None = None,
     ) -> None:
         """Plot results for a specific category
 
@@ -2886,6 +2898,8 @@ def __plot_result(
             rdata: Result data
             objs: List of objects
             param: Plot result parameters. If None, show dialog to get parameters.
+            result_group_id: UUID of the group to add result signals to. If None,
+             add to current group.
         """
         # Regrouping ResultShape results by their `title` attribute:
         grouped_results: dict[str, list[GeometryAdapter | TableAdapter]] = {}
@@ -2940,6 +2954,18 @@ def __plot_result(
                     )
                 return
             obj = objs[0]
+            # Build a string with source object short IDs (max 3, then use "...")
+            max_ids_to_show = 3
+            short_ids = [get_short_id(obj) for obj in objs]
+            if len(short_ids) <= max_ids_to_show:
+                source_ids = ", ".join(short_ids)
+            else:
+                # Show first 2, "...", then last one: "s001, s002, ..., s010"
+                source_ids = (
+                    ", ".join(short_ids[: max_ids_to_show - 1])
+                    + ", ..., "
+                    + short_ids[-1]
+                )
             for i_roi in all_roi_indexes[0]:
                 roi_suffix = f"|ROI{int(i_roi + 1)}" if i_roi >= 0 else ""
                 for title, results in grouped_results.items():  # title
@@ -2953,7 +2979,12 @@ def __plot_result(
                     if i_roi >= 0:
                         roi_suffix = f"|{obj.roi.get_single_roi_title(int(i_roi))}"
                     self.__add_result_signal(
-                        x, y, f"{title}{roi_suffix}", param.xaxis, param.yaxis
+                        x,
+                        y,
+                        f"{title} ({source_ids}){roi_suffix}",
+                        param.xaxis,
+                        param.yaxis,
+                        result_group_id,
                     )
         else:
             # One curve per result title, per object and per ROI
@@ -2974,7 +3005,9 @@ def __plot_result(
                         y = roi_data[param.yaxis].values
                         shid = get_short_id(objs[index])
                         stitle = f"{title} ({shid}){roi_suffix}"
-                        self.__add_result_signal(x, y, stitle, param.xaxis, param.yaxis)
+                        self.__add_result_signal(
+                            x, y, stitle, param.xaxis, param.yaxis, result_group_id
+                        )
 
     def plot_results(
         self,
@@ -2993,6 +3026,17 @@ def plot_results(
         objs = self.objview.get_sel_objects(include_groups=True)
         rdatadict = create_resultdata_dict(objs)
         if rdatadict:
+            # Always use or create a "Results" group for all plot results
+            rgroup_title = _("Results")
+            target_panel = self.mainwindow.signalpanel
+            try:
+                # Check if a "Results" group already exists in the signal panel
+                rgroup = target_panel.objmodel.get_group_from_title(rgroup_title)
+            except KeyError:
+                # Create the group if it doesn't exist
+                rgroup = target_panel.add_group(rgroup_title)
+            result_group_id = get_uuid(rgroup)
+
             for category, rdata in rdatadict.items():
                 param = None
                 if kind is not None and xaxis is not None and yaxis is not None:
@@ -3003,7 +3047,7 @@ def plot_results(
                     param.xaxis = xaxis
                     param.yaxis = yaxis
 
-                self.__plot_result(category, rdata, objs, param)
+                self.__plot_result(category, rdata, objs, param, result_group_id)
         else:
             self.__show_no_result_warning()
 

datalab/tests/features/common/plot_results_app_test.py

@@ -9,6 +9,7 @@
   - Results with and without ROIs
   - Both signal and image panels
   - Multiple result types (scalar and geometry results)
+  - Group selection creates a new result group
 """
 
 # guitest: show
@@ -21,7 +22,9 @@
 import sigima.params
 from sigima.tests import data as test_data
 
+from datalab.config import _ as translate
 from datalab.env import execenv
+from datalab.objectmodel import get_uuid
 from datalab.tests import datalab_test_app_context
 
 
@@ -46,6 +49,7 @@ def test_plot_results_signals_one_curve_per_title():
     """Test plot results feature with signals, one curve per title.
 
     Create signals with single-value results (e.g., FWHM) and plot them.
+    Verify that results are created in the "Results" group.
     """
     with datalab_test_app_context() as win:
         panel = win.signalpanel
@@ -64,10 +68,34 @@ def test_plot_results_signals_one_curve_per_title():
                 # (trade-off for noise robustness)
                 panel.processor.run_feature("fwhm", sigima.params.FWHMParam())
 
+            # Get number of groups before plotting
+            groups_before = len(panel.objmodel.get_groups())
+
             panel.objview.selectAll()
             panel.show_results()
             panel.plot_results(kind="one_curve_per_title", xaxis="indices", yaxis="Δx")
 
+            # Verify a Results group was created
+            groups_after = panel.objmodel.get_groups()
+            assert len(groups_after) == groups_before + 1, (
+                f"Expected {groups_before + 1} groups, got {len(groups_after)}"
+            )
+
+            # Verify the new group is named "Results"
+            from datalab.config import _ as translate
+
+            expected_title = translate("Results")
+            result_group = groups_after[-1]
+            assert result_group.title == expected_title, (
+                f"Expected last group to be '{expected_title}', "
+                f"got '{result_group.title}'"
+            )
+
+            # Verify the Results group contains the result signal
+            assert len(result_group) > 0, (
+                "Results group should contain at least one result signal"
+            )
+
             fwhm_var_th = sigima.objects.create_signal("FWHM_Theoretical", x_th, y_th)
             panel.add_object(fwhm_var_th)
             panel.objview.select_objects((6, 7))
@@ -124,7 +152,123 @@ def test_plot_results_images_with_rois():
             panel.plot_results(kind="one_curve_per_title", xaxis="indices", yaxis="x")
 
 
+def test_plot_results_with_group_selection():
+    """Test plot results with Results group.
+
+    All plot results operations should create result signals in a reusable
+    "Results" group for better organization.
+    """
+    with datalab_test_app_context() as win:
+        panel = win.signalpanel
+
+        with execenv.context(unattended=True):
+            # Create a group with signals
+            panel.add_group("Test Group")
+
+            # Add signals and compute FWHM
+            for i, (sig, _) in enumerate(iterate_noisy_signals(3, a=10.0, sigma=0.01)):
+                sig.title = f"Signal_{i + 1}"
+                panel.add_object(sig)
+                panel.processor.run_feature("fwhm", sigima.params.FWHMParam())
+
+            # Get the number of groups before plotting results
+            groups_before = len(panel.objmodel.get_groups())
+
+            # Select the group (not individual objects)
+            panel.objview.select_groups([1])
+
+            # Verify the group is selected
+            sel_groups = panel.objview.get_sel_groups()
+            assert len(sel_groups) == 1, (
+                f"Expected 1 selected group, got {len(sel_groups)}"
+            )
+
+            # Plot results - this should create or reuse a "Results" group
+            panel.plot_results(kind="one_curve_per_title", xaxis="indices", yaxis="Δx")
+
+            # Verify a new group was created
+            groups_after = panel.objmodel.get_groups()
+            assert len(groups_after) == groups_before + 1, (
+                f"Expected {groups_before + 1} groups, got {len(groups_after)}"
+            )
+
+            # Check that the new group is named "Results" (or its translation)
+            expected_title = translate("Results")
+            result_group = groups_after[-1]  # Last group should be Results
+            assert result_group.title == expected_title, (
+                f"Expected last group to be '{expected_title}', "
+                f"got '{result_group.title}'"
+            )
+
+            # Check that the Results group contains at least one result signal
+            assert len(result_group) > 0, (
+                "Results group should contain at least one result signal"
+            )
+
+            # Verify that the result signal title includes source object short IDs
+            result_signal = list(result_group)[0]
+            # Should contain all three source signal IDs: s001, s002, s003
+            # (s000 is the default group, so signals start at s001)
+            assert "(s001, s002, s003)" in result_signal.title, (
+                f"Result signal title should include source IDs (s001, s002, s003), "
+                f"got '{result_signal.title}'"
+            )
+
+            # Test that the group is reused: create another group and plot results
+            test_group_2 = panel.add_group("Test Group 2")
+            test_group_2_id = get_uuid(test_group_2)
+            for i, (sig, _) in enumerate(iterate_noisy_signals(2, a=10.0, sigma=0.01)):
+                sig.title = f"Signal2_{i + 1}"
+                panel.add_object(sig, group_id=test_group_2_id)
+                panel.processor.run_feature("fwhm", sigima.params.FWHMParam())
+
+            # Select the second group and plot results again
+            panel.objview.select_groups([3])
+
+            # Plot results again
+            num_results_before = len(result_group)
+            panel.plot_results(kind="one_curve_per_title", xaxis="indices", yaxis="Δx")
+
+            # Verify no new group was created (reused existing Results group)
+            groups_final = panel.objmodel.get_groups()
+            assert len(groups_final) == len(groups_after), (
+                "Results group should be reused, no new group created"
+            )
+
+            # Verify more results were added to the existing Results group
+            assert len(result_group) > num_results_before, (
+                "More results should be added to the existing Results group"
+            )
+
+            # Test with many objects (more than 3) to verify "..." is used
+            test_group_3 = panel.add_group("Test Group 3")
+            test_group_3_id = get_uuid(test_group_3)
+            for i, (sig, _) in enumerate(iterate_noisy_signals(5, a=10.0, sigma=0.01)):
+                sig.title = f"Signal3_{i + 1}"
+                panel.add_object(sig, group_id=test_group_3_id)
+                panel.processor.run_feature("fwhm", sigima.params.FWHMParam())
+
+            # Select the third group
+            panel.objview.select_groups([4])
+
+            # Plot results
+            num_results_before = len(result_group)
+            panel.plot_results(kind="one_curve_per_title", xaxis="indices", yaxis="Δx")
+
+            # Verify the result signal title uses "..." for many objects
+            new_results = list(result_group)[num_results_before:]
+            assert len(new_results) > 0, "Should have new results"
+            result_signal_many = new_results[0]
+            # With 5 source signals, should show first 2 IDs, "...", then last ID
+            # Format: "fwhm (s..., s..., ..., s...): ..."
+            assert ", ..., " in result_signal_many.title, (
+                f"Result signal title should use '...' before last ID, "
+                f"got '{result_signal_many.title}'"
+            )
+
+
 if __name__ == "__main__":
     test_plot_results_signals_one_curve_per_title()
     test_plot_results_images_one_curve_per_object()
     test_plot_results_images_with_rois()
+    test_plot_results_with_group_selection()

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-07 18:27+0100\n"
+"POT-Creation-Date: 2025-11-09 09:59+0100\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language: fr\n"
@@ -159,6 +159,39 @@ msgstr "Le flux de travail basé sur le presse-papiers permet une réutilisation
 msgid "Independent from ROI management - annotations and ROIs can coexist on same object"
 msgstr "Indépendant de la gestion des ROI - les annotations et les ROI peuvent coexister sur le même objet"
 
+msgid "**Plot results improvements**: Enhanced result plotting with better organization and informative titles"
+msgstr "**Améliorations du tracé des résultats** : Tracé des résultats amélioré avec une meilleure organisation et des titres informatifs"
+
+msgid "**Results group creation**: All plot results operations now automatically organize results in a dedicated \"Results\" group"
+msgstr "**Création de groupe de résultats** : Toutes les opérations de tracé des résultats organisent désormais automatiquement les résultats dans un groupe \"Résultats\" dédié"
+
+msgid "First time plotting results creates a new \"Results\" group in the signal panel"
+msgstr "La première fois que les résultats sont tracés, un nouveau groupe \"Résultats\" est créé dans le panneau des signaux"
+
+msgid "Subsequent result plots reuse the same \"Results\" group for better organization"
+msgstr "Les tracés de résultats suivants réutilisent le même groupe \"Résultats\" pour une meilleure organisation"
+
+msgid "Keeps workspace clean by consolidating all results in one well-defined location"
+msgstr "Garde l'espace de travail propre en consolidant tous les résultats en un seul endroit bien défini"
+
+msgid "Consistent behavior regardless of whether groups or individual objects are selected"
+msgstr "Comportement cohérent, que des groupes ou des objets individuels soient sélectionnés"
+
+msgid "**Comprehensive result titles**: Result signal titles now include source object identifiers"
+msgstr "**Titres de résultats complets** : Les titres des signaux de résultats incluent désormais les identifiants des objets source"
+
+msgid "For \"one curve per title\" mode: titles include all source object short IDs (e.g., \"FWHM (s001, s002, s003): Δx = f(indices)\")"
+msgstr "Pour le mode \"une courbe par titre\" : les titres incluent tous les identifiants courts des objets source (par exemple, \"FWHM (s001, s002, s003): Δx = f(indices)\")"
+
+msgid "For many source objects (>3): uses compact format showing first 2 IDs, \"...\", then last ID (e.g., \"FWHM (s001, s002, ..., s010): Δx = f(indices)\")"
+msgstr "Pour de nombreux objets source (>3) : utilise un format compact montrant les 2 premiers identifiants, \"...\", puis le dernier identifiant (par exemple, \"FWHM (s001, s002, ..., s010): Δx = f(indices)\")"
+
+msgid "Makes it easy to identify which objects contributed to plotted results"
+msgstr "Facilite l'identification des objets ayant contribué aux résultats tracés"
+
+msgid "Maintains consistency with \"one curve per object\" mode which already included IDs"
+msgstr "Maintient la cohérence avec le mode \"une courbe par objet\" qui incluait déjà des identifiants"
+
 msgid "**Metadata management improvements**: Enhanced metadata handling with clipboard-based workflow"
 msgstr "**Améliorations de la gestion des métadonnées** : Gestion des métadonnées améliorée avec un flux de travail basé sur le presse-papiers"