action: use to_human_readable_repr

Author: james-d-mitchell

src/action.cpp

@@ -106,7 +106,9 @@ In this documentation we refer to:
   553
 )pbdoc");
 
-      thing.def("__repr__", &detail::to_string<Action_ const&>);
+      thing.def("__repr__", [](Action_ const& self) {
+        return to_human_readable_repr(self);
+      });
       thing.def("__getitem__", &Action_::at, py::is_operator(), py::arg("pos"));
 
       // No doc since not used directly, but is used indirectly in the

src/libsemigroups_pybind11/action.py

@@ -180,7 +180,9 @@ class Action(_CxxWrapper):  # pylint: disable=missing-class-docstring
     ########################################################################
 
     # pylint: disable=redefined-outer-name, line-too-long
-    def __init__(self: _Self, *args, generators=None, seeds=None, func=None, side=None) -> None:
+    def __init__(
+        self: _Self, *args, generators=None, seeds=None, func=None, side=None
+    ) -> None:
         """
         :sig=(self: Action, generators:list[Element] = None, seeds:list[Point] = None, func:Callable[[Point, Element], Point] = None, side:side = None) -> None:
 
@@ -212,7 +214,9 @@ def __init__(self: _Self, *args, generators=None, seeds=None, func=None, side=No
         if _to_cxx(self) is not None:
             return
         if len(args) != 0:
-            raise TypeError(f"expected 0 positional arguments, but found {len(args)}")
+            raise TypeError(
+                f"expected 0 positional arguments, but found {len(args)}"
+            )
         if not isinstance(generators, list):
             raise TypeError(
                 "expected the keyword argument 'generators' to be "
@@ -275,7 +279,9 @@ def generators(self: _Self) -> Iterator[Element]:
 
 _copy_cxx_mem_fns(_RightActionPPerm1PPerm1, Action)
 
-for _type in (
+for (
+    _type
+) in (
     Action._py_template_params_to_cxx_type.values()  # pylint: disable=protected-access
 ):
     _register_cxx_wrapped_type(_type, Action)

src/libsemigroups_pybind11/adapters.py

@@ -51,7 +51,9 @@ class _ImageAction(_CxxWrapper):
     Element = _TypeVar("Element")
     Point = _TypeVar("Point")
 
-    def __init__(self: _Self, *args, point: Point = None, element: Element = None) -> None:
+    def __init__(
+        self: _Self, *args, point: Point = None, element: Element = None
+    ) -> None:
         """
         Construct from sample element and sample point.
 
@@ -71,7 +73,9 @@ def __init__(self: _Self, *args, point: Point = None, element: Element = None) -
         if _to_cxx(self) is not None:
             return
         if len(args) != 0:
-            raise TypeError(f"expected 0 positional arguments, but found {len(args)}")
+            raise TypeError(
+                f"expected 0 positional arguments, but found {len(args)}"
+            )
         self.py_template_params = (
             type(_to_cxx(element)),
             type(_to_cxx(point)),
@@ -119,7 +123,9 @@ def __call__(self: _Self, pt: Point, x: Element) -> Point:
 
 _copy_cxx_mem_fns(_ImageRightActionPPerm1PPerm1, ImageRightAction)
 
-for _type in (
+for (
+    _type
+) in (
     ImageRightAction._py_template_params_to_cxx_type.values()  # pylint:disable=protected-access
 ):
     _register_cxx_wrapped_type(_type, ImageRightAction)

tests/test_action.py

@@ -367,9 +367,7 @@ def test_action_coverage():
         seeds=[PPerm.one(17)],
     )
 
-    assert (
-        repr(right) == "<incomplete right action with 1 generators, 1 points>"
-    )
+    assert repr(right) == "<incomplete right action with 1 generator, 1 point>"
     with pytest.raises(TypeError):
         right.add_generator(BMat8(0))