Changed select_groups first argument selection

Author: PierreRaybaut

CHANGELOG.md

@@ -8,6 +8,7 @@ Changes:
   * Added dict-like interface (e.g. `proxy['obj123']`)
   * Renamed `switch_to_panel` method to `set_current_panel` (compatibility with DataLab 0.9)
   * Added `get_current_panel` method
+  * Changed `select_groups` first argument `selection` (compatibility with DataLab 0.9)
 
 ## Version 0.3.0 ##
 

cdlclient/baseproxy.py

@@ -47,15 +47,15 @@ def __iter__(self) -> Iterator[SignalObj | ImageObj]:
         for uuid in uuids:
             yield self.get_object(uuid)
 
-    def __repr__(self) -> str:
-        """Return object representation"""
-        return self.__str__()
-
     def __str__(self) -> str:
         """Return object string representation"""
+        return super().__repr__()
+
+    def __repr__(self) -> str:
+        """Return object representation"""
         titles = self.get_object_titles()
         uuids = self.get_object_uuids()
-        text = f"{self.__class__.__name__} (DataLab instance, {len(titles)} items):\n"
+        text = f"{str(self)} (DataLab, {len(titles)} items):\n"
         for uuid, title in zip(uuids, titles):
             text += f"  {uuid}: {title}\n"
         return text
@@ -249,12 +249,13 @@ def select_objects(
 
     @abc.abstractmethod
     def select_groups(
-        self, selection: list[int | str], panel: str | None = None
+        self, selection: list[int | str] | None = None, panel: str | None = None
     ) -> None:
         """Select groups in current panel.
 
         Args:
-            selection (list[int | str]): List of group numbers or uuids to select
+            selection: List of group numbers (1 to N), or list of group uuids,
+             or None to select all groups. Defaults to None.
             panel (str | None): panel name (valid values: "signal", "image").
                 If None, current panel is used. Defaults to None.
         """
@@ -525,12 +526,13 @@ def select_objects(
         self._cdl.select_objects(selection, group_num, panel)
 
     def select_groups(
-        self, selection: list[int | str], panel: str | None = None
+        self, selection: list[int | str] | None = None, panel: str | None = None
     ) -> None:
         """Select groups in current panel.
 
         Args:
-            selection (list[int | str]): List of group numbers or uuids to select
+            selection: List of group numbers (1 to N), or list of group uuids,
+             or None to select all groups. Defaults to None.
             panel (str | None): panel name (valid values: "signal", "image").
                 If None, current panel is used. Defaults to None.
         """

cdlclient/tests/remoteclient_unit.py

@@ -56,7 +56,7 @@ def multiple_commands(remote: SimpleRemoteProxy):
         assert len(items) == 1 and items[0].get_rect() == area
         execenv.print("OK")
         remote.add_label_with_title(f"Image uuid: {uuid}")
-        remote.select_groups([0])
+        remote.select_groups([1])
         remote.select_objects([uuid])
         remote.delete_metadata()