feat: workflow enhancements for better tool results (#1723)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com>
Co-authored-by: Roberto Pastor Muela <37798125+RobPasMue@users.noreply.github.com>
Co-authored-by: rward <ryan.ward@ansys.com>
Co-authored-by: Ryan Ward <129954471+RyanJWard@users.noreply.github.com>

Author: 5 people

doc/changelog.d/1723.added.md

@@ -0,0 +1 @@
+workflow enhancements for better tool results

src/ansys/geometry/core/designer/geometry_commands.py

@@ -35,6 +35,7 @@
     ExtrudeFacesUpToRequest,
     FilletRequest,
     FullFilletRequest,
+    ModifyCircularPatternRequest,
     ModifyLinearPatternRequest,
     PatternRequest,
     RenameObjectRequest,
@@ -744,6 +745,58 @@ def create_circular_pattern(
 
         return result.result.success
 
+    @protect_grpc
+    @min_backend_version(25, 2, 0)
+    def modify_circular_pattern(
+        self,
+        selection: Union["Face", list["Face"]],
+        circular_count: int = 0,
+        linear_count: int = 0,
+        step_angle: Real = 0.0,
+        step_linear: Real = 0.0,
+    ) -> bool:
+        """Modify a circular pattern. Leave an argument at 0 for it to remain unchanged.
+
+        Parameters
+        ----------
+        selection : Face | list[Face]
+            Faces that belong to the pattern.
+        circular_count : int, default: 0
+            How many members are in the circular pattern.
+        linear_count : int, default: 0
+            How many times the circular pattern repeats along the radial lines for a
+            two-dimensional pattern.
+        step_angle : Real, default: 0.0
+            Defines the circular angle.
+        step_linear : Real, default: 0.0
+            Defines the step, along the radial lines, for a pattern dimension greater than 1.
+
+        Returns
+        -------
+        bool
+            ``True`` when successful, ``False`` when failed.
+        """
+        from ansys.geometry.core.designer.face import Face
+
+        selection: list[Face] = selection if isinstance(selection, list) else [selection]
+
+        check_type_all_elements_in_iterable(selection, Face)
+
+        for object in selection:
+            object.body._reset_tessellation_cache()
+
+        result = self._commands_stub.ModifyCircularPattern(
+            ModifyCircularPatternRequest(
+                selection=[object._grpc_id for object in selection],
+                circular_count=circular_count,
+                linear_count=linear_count,
+                step_angle=step_angle,
+                step_linear=step_linear,
+            )
+        )
+
+        return result.result.success
+
     @protect_grpc
     @min_backend_version(25, 2, 0)
     def create_fill_pattern(

src/ansys/geometry/core/tools/prepare_tools.py

@@ -41,6 +41,7 @@
     get_design_from_face,
 )
 from ansys.geometry.core.misc.checks import check_type_all_elements_in_iterable, min_backend_version
+from ansys.geometry.core.tools.repair_tool_message import RepairToolMessage
 from ansys.geometry.core.typing import Real
 
 if TYPE_CHECKING:  # pragma: no cover
@@ -204,3 +205,49 @@ def share_topology(
             )
         )
         return share_topo_response.result
+
+    @protect_grpc
+    @min_backend_version(25, 2, 0)
+    def enhanced_share_topology(
+        self, bodies: list["Body"], tol: Real = 0.0, preserve_instances: bool = False
+    ) -> RepairToolMessage:
+        """Share topology between the chosen bodies.
+
+        Parameters
+        ----------
+        bodies : list[Body]
+            List of bodies to share topology between.
+        tol : Real
+            Maximum distance between bodies.
+        preserve_instances : bool
+            Whether instances are preserved.
+
+        Returns
+        -------
+        RepairToolMessage
+            Message containing number of problem areas found/fixed, created and/or modified bodies.
+        """
+        from ansys.geometry.core.designer.body import Body
+
+        if not bodies:
+            return RepairToolMessage(False, [], [], 0, 0)
+
+        # Verify inputs
+        check_type_all_elements_in_iterable(bodies, Body)
+
+        share_topo_response = self._prepare_stub.EnhancedShareTopology(
+            ShareTopologyRequest(
+                selection=[GRPCBody(id=body.id) for body in bodies],
+                tolerance=DoubleValue(value=tol),
+                preserve_instances=BoolValue(value=preserve_instances),
+            )
+        )
+
+        message = RepairToolMessage(
+            share_topo_response.success,
+            share_topo_response.created_bodies_monikers,
+            share_topo_response.modified_bodies_monikers,
+            share_topo_response.found,
+            share_topo_response.repaired,
+        )
+        return message

src/ansys/geometry/core/tools/repair_tool_message.py

@@ -25,7 +25,14 @@
 class RepairToolMessage:
     """Provides return message for the repair tool methods."""
 
-    def __init__(self, success: bool, created_bodies: list[str], modified_bodies: list[str]):
+    def __init__(
+        self,
+        success: bool,
+        created_bodies: list[str],
+        modified_bodies: list[str],
+        found: int = -1,
+        repaired: int = -1,
+    ):
         """Initialize a new instance of the extra edge problem area class.
 
         Parameters
@@ -36,10 +43,20 @@ def __init__(self, success: bool, created_bodies: list[str], modified_bodies: li
             List of bodies created after the repair operation.
         modified_bodies: list[str]
             List of bodies modified after the repair operation.
+        found: int, default: -1
+            Number of problem areas found for the repair operation.
+            If default, the operation does not provide the number of found problem areas.
+        repaired: int, default: -1
+            Number of problem areas repaired during the repair operation.
+            If default, the operation does not provide the number of fixed problem areas.
+
+
         """
         self._success = success
         self._created_bodies = created_bodies
         self._modified_bodies = modified_bodies
+        self._found = found
+        self._repaired = repaired
 
     @property
     def success(self) -> bool:
@@ -55,3 +72,13 @@ def created_bodies(self) -> list[str]:
     def modified_bodies(self) -> list[str]:
         """The list of the modified bodies after the repair operation."""
         return self._modified_bodies
+
+    @property
+    def found(self) -> int:
+        """Number of problem areas found for the repair operation."""
+        return self._found
+
+    @property
+    def repaired(self) -> int:
+        """Number of problem areas repaired during the repair operation."""
+        return self._repaired

src/ansys/geometry/core/tools/repair_tools.py

@@ -470,7 +470,7 @@ def find_interferences(
     @protect_grpc
     @min_backend_version(25, 2, 0)
     def find_and_fix_short_edges(
-        self, bodies: list["Body"], length: Real = 0.0
+        self, bodies: list["Body"], length: Real = 0.0, comprehensive_result: bool = False
     ) -> RepairToolMessage:
         """Find and fix the short edge problem areas.
 
@@ -484,24 +484,29 @@ def find_and_fix_short_edges(
             List of bodies that short edges are investigated on.
         length : Real, optional
             The maximum length of the edges. By default, 0.0.
+        comprehensive_result : bool, optional
+            Whether to fix all problem areas individually.
+            By default, False.
 
         Returns
         -------
         RepairToolMessage
-            Message containing created and/or modified bodies.
+            Message containing number of problem areas found/fixed, created and/or modified bodies.
         """
         from ansys.geometry.core.designer.body import Body
 
         check_type_all_elements_in_iterable(bodies, Body)
         check_type(length, Real)
+        check_type(comprehensive_result, bool)
 
         if not bodies:
-            return RepairToolMessage(False, [], [])
+            return RepairToolMessage(False, [], [], 0, 0)
 
         response = self._repair_stub.FindAndFixShortEdges(
             FindShortEdgesRequest(
                 selection=[body.id for body in bodies],
                 max_edge_length=DoubleValue(value=length),
+                comprehensive=comprehensive_result,
             )
         )
 
@@ -511,12 +516,16 @@ def find_and_fix_short_edges(
             response.success,
             response.created_bodies_monikers,
             response.modified_bodies_monikers,
+            response.found,
+            response.repaired,
         )
         return message
 
     @protect_grpc
     @min_backend_version(25, 2, 0)
-    def find_and_fix_extra_edges(self, bodies: list["Body"]) -> RepairToolMessage:
+    def find_and_fix_extra_edges(
+        self, bodies: list["Body"], comprehensive_result: bool = False
+    ) -> RepairToolMessage:
         """Find and fix the extra edge problem areas.
 
         Notes
@@ -529,22 +538,26 @@ def find_and_fix_extra_edges(self, bodies: list["Body"]) -> RepairToolMessage:
             List of bodies that short edges are investigated on.
         length : Real
             The maximum length of the edges.
+        comprehensive_result : bool, optional
+            Whether to fix all problem areas individually.
+            By default, False.
 
         Returns
         -------
         RepairToolMessage
-            Message containing created and/or modified bodies.
+            Message containing number of problem areas found/fixed, created and/or modified bodies.
         """
         from ansys.geometry.core.designer.body import Body
 
         check_type_all_elements_in_iterable(bodies, Body)
+        check_type(comprehensive_result, bool)
 
         if not bodies:
-            return RepairToolMessage(False, [], [])
+            return RepairToolMessage(False, [], [], 0, 0)
 
         response = self._repair_stub.FindAndFixExtraEdges(
             FindExtraEdgesRequest(
-                selection=[body.id for body in bodies],
+                selection=[body.id for body in bodies], comprehensive=comprehensive_result
             )
         )
 
@@ -554,13 +567,19 @@ def find_and_fix_extra_edges(self, bodies: list["Body"]) -> RepairToolMessage:
             response.success,
             response.created_bodies_monikers,
             response.modified_bodies_monikers,
+            response.found,
+            response.repaired,
         )
         return message
 
     @protect_grpc
     @min_backend_version(25, 2, 0)
     def find_and_fix_split_edges(
-        self, bodies: list["Body"], angle: Real = 0.0, length: Real = 0.0
+        self,
+        bodies: list["Body"],
+        angle: Real = 0.0,
+        length: Real = 0.0,
+        comprehensive_result: bool = False,
     ) -> RepairToolMessage:
         """Find and fix the split edge problem areas.
 
@@ -576,28 +595,35 @@ def find_and_fix_split_edges(
             The maximum angle between edges. By default, 0.0.
         length : Real, optional
             The maximum length of the edges. By default, 0.0.
+        comprehensive_result : bool, optional
+            Whether to fix all problem areas individually.
+            By default, False.
 
         Returns
         -------
         RepairToolMessage
-            Message containing created and/or modified bodies.
+            Message containing number of problem areas found/fixed, created and/or modified bodies.
         """
         from ansys.geometry.core.designer.body import Body
 
         check_type_all_elements_in_iterable(bodies, Body)
         check_type(angle, Real)
         check_type(length, Real)
+        check_type(comprehensive_result, bool)
 
         if not bodies:
-            return RepairToolMessage(False, [], [])
+            return RepairToolMessage(False, [], [], 0, 0)
 
         angle_value = DoubleValue(value=float(angle))
         length_value = DoubleValue(value=float(length))
         body_ids = [body.id for body in bodies]
 
         response = self._repair_stub.FindAndFixSplitEdges(
             FindSplitEdgesRequest(
-                bodies_or_faces=body_ids, angle=angle_value, distance=length_value
+                bodies_or_faces=body_ids,
+                angle=angle_value,
+                distance=length_value,
+                comprehensive=comprehensive_result,
             )
         )
 
@@ -607,6 +633,57 @@ def find_and_fix_split_edges(
             response.success,
             response.created_bodies_monikers,
             response.modified_bodies_monikers,
+            response.found,
+            response.repaired,
+        )
+        return message
+
+    @protect_grpc
+    @min_backend_version(25, 2, 0)
+    def find_and_fix_simplify(
+        self, bodies: list["Body"], comprehensive_result: bool = False
+    ) -> RepairToolMessage:
+        """Find and simplify the provided geometry.
+
+        Notes
+        -----
+        This method simplifies the provided geometry.
+
+        Parameters
+        ----------
+        bodies : list[Body]
+            List of bodies to be simplified.
+        comprehensive_result : bool, optional
+            Whether to fix all problem areas individually.
+            By default, False.
+
+        Returns
+        -------
+        RepairToolMessage
+            Message containing number of problem areas found/fixed, created and/or modified bodies.
+        """
+        from ansys.geometry.core.designer.body import Body
+
+        check_type_all_elements_in_iterable(bodies, Body)
+        check_type(comprehensive_result, bool)
+
+        if not bodies:
+            return RepairToolMessage(False, [], [], 0, 0)
+
+        body_ids = [body.id for body in bodies]
+
+        response = self._repair_stub.FindAndSimplify(
+            FindAdjustSimplifyRequest(selection=body_ids, comprehensive=comprehensive_result)
+        )
+
+        parent_design = get_design_from_body(bodies[0])
+        parent_design._update_design_inplace()
+        message = RepairToolMessage(
+            response.success,
+            response.created_bodies_monikers,
+            response.modified_bodies_monikers,
+            response.found,
+            response.repaired,
         )
         return message