chore: v1 implementation of repair stubs

doc/changelog.d/1753.maintenance.md

@@ -0,0 +1 @@
+update CHANGELOG for v0.9.0

doc/changelog.d/2441.maintenance.md

@@ -0,0 +1 @@
+Chore: v1 implementation of repair stubs

src/ansys/geometry/core/_grpc/_services/v0/repair_tools.py

@@ -19,12 +19,7 @@
 # LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
-"""Module containing the repair tools service implementation for v0.
-
-This module defines an abstract base class for a gRPC-based repair tools service.
-The class provides a set of abstract methods for identifying and repairing various
-geometry issues, such as split edges, extra edges, duplicate faces etc.
-"""
+"""Module containing the repair tools service implementation for v0."""
 
 from google.protobuf.wrappers_pb2 import Int32Value
 import grpc
@@ -184,27 +179,11 @@ def find_missing_faces(self, **kwargs) -> dict:  # noqa: D102
         from ansys.api.geometry.v0.repairtools_pb2 import FindMissingFacesRequest
         from google.protobuf.wrappers_pb2 import DoubleValue
 
-        from ansys.geometry.core.logger import LOG
-
         from ..base.conversions import (
             from_measurement_to_server_angle,
             from_measurement_to_server_length,
         )
 
-        # Check the backend version to set optional parameters
-        if kwargs["backend_version"] < (26, 1, 0) and (
-            kwargs["angle"] is not None or kwargs["distance"] is not None
-        ):
-            # If the backend version is less than 26.1.0, set angle and distance to None
-            kwargs["angle"] = None
-            kwargs["distance"] = None
-
-            # Log a warning
-            LOG.warning(
-                "The backend version is less than 26.1.0, so angle and distance parameters will be"
-                "ignored. Please update the backend to use these parameters."
-            )
-
         # Create the request - assumes all inputs are valid and of the proper type
         request = FindMissingFacesRequest(
             faces=kwargs["faces"],
@@ -235,27 +214,11 @@ def find_small_faces(self, **kwargs) -> dict:  # noqa: D102
         from ansys.api.geometry.v0.repairtools_pb2 import FindSmallFacesRequest
         from google.protobuf.wrappers_pb2 import DoubleValue
 
-        from ansys.geometry.core.logger import LOG
-
         from ..base.conversions import (
             from_measurement_to_server_area,
             from_measurement_to_server_length,
         )
 
-        # Check the backend version to set optional parameters
-        if kwargs["backend_version"] < (26, 1, 0) and (
-            kwargs["area"] is not None or kwargs["width"] is not None
-        ):
-            # If the backend version is less than 26.1.0, set area and width to None
-            kwargs["area"] = None
-            kwargs["width"] = None
-
-            # Log a warning
-            LOG.warning(
-                "The backend version is less than 26.1.0, so area and width parameters will be"
-                "ignored. Please update the backend to use these parameters."
-            )
-
         # Create the request - assumes all inputs are valid and of the proper type
         request = FindSmallFacesRequest(
             selection=kwargs["selection"],
@@ -286,17 +249,8 @@ def find_stitch_faces(self, **kwargs) -> dict:  # noqa: D102
         from ansys.api.geometry.v0.repairtools_pb2 import FindStitchFacesRequest
         from google.protobuf.wrappers_pb2 import DoubleValue
 
-        from ansys.geometry.core.logger import LOG
-
         from ..base.conversions import from_measurement_to_server_length
 
-        if kwargs["backend_version"] < (26, 1, 0) and kwargs["distance"] is not None:
-            # If the backend version is less than 26.1.0, set distance to None and log warning
-            kwargs["distance"] = None
-            LOG.warning(
-                "The backend version is less than 26.1.0, so distance parameter will be ignored. "
-            )
-
         # Create the request - assumes all inputs are valid and of the proper type
         request = FindStitchFacesRequest(
             faces=kwargs["faces"],
@@ -393,7 +347,7 @@ def find_and_fix_short_edges(self, **kwargs):  # noqa: D102
             "repaired": response.repaired,
             "created_bodies_monikers": [],
             "modified_bodies_monikers": [],
-            "complete_command_response": serialized_tracker_response,
+            "tracker_response": serialized_tracker_response,
         }
 
     @protect_grpc
@@ -420,7 +374,7 @@ def find_and_fix_extra_edges(self, **kwargs) -> dict:  # noqa: D102
             "repaired": response.repaired,
             "created_bodies_monikers": response.created_bodies_monikers,
             "modified_bodies_monikers": response.modified_bodies_monikers,
-            "complete_command_response": serialized_tracker_response,
+            "tracker_response": serialized_tracker_response,
         }
 
     @protect_grpc
@@ -449,7 +403,7 @@ def find_and_fix_split_edges(self, **kwargs) -> dict:  # noqa: D102
             "repaired": response.repaired,
             "created_bodies_monikers": [],
             "modified_bodies_monikers": [],
-            "complete_command_response": serialized_tracker_response,
+            "tracker_response": serialized_tracker_response,
         }
 
     @protect_grpc
@@ -476,7 +430,7 @@ def find_and_fix_simplify(self, **kwargs) -> dict:  # noqa: D102
             "repaired": response.repaired,
             "created_bodies_monikers": [],
             "modified_bodies_monikers": [],
-            "complete_command_response": serialized_tracker_response,
+            "tracker_response": serialized_tracker_response,
         }
 
     @protect_grpc
@@ -514,7 +468,7 @@ def find_and_fix_stitch_faces(self, **kwargs) -> dict:  # noqa: D102
             "modified_bodies_monikers": response.modified_bodies_monikers,
             "found": response.found,
             "repaired": response.repaired,
-            "complete_command_response": serialized_tracker_response,
+            "tracker_response": serialized_tracker_response,
         }
 
     @protect_grpc
@@ -566,7 +520,9 @@ def fix_duplicate_faces(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -590,7 +546,9 @@ def fix_missing_faces(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -614,7 +572,9 @@ def fix_inexact_edges(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -636,7 +596,9 @@ def fix_extra_edges(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -658,7 +620,9 @@ def fix_short_edges(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -680,7 +644,9 @@ def fix_small_faces(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -702,7 +668,9 @@ def fix_split_edges(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -726,7 +694,9 @@ def fix_stitch_faces(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -750,7 +720,9 @@ def fix_unsimplified_faces(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     @protect_grpc
@@ -774,7 +746,9 @@ def fix_interference(self, **kwargs) -> dict:  # noqa: D102
         # Return the response - formatted as a dictionary
         return {
             "tracker_response": serialized_tracker_response,
-            "repair_tracker_response": self.__serialize_message_response(response),
+            "success": response.result.success,
+            "created_bodies_monikers": response.result.created_bodies_monikers,
+            "modified_bodies_monikers": response.result.modified_bodies_monikers,
         }
 
     def __serialize_inspect_result_response(self, response) -> dict:  # noqa: D102
@@ -830,10 +804,3 @@ def serialize_issue(issue):
                 for body_issues in response.issues_by_body
             ]
         }
-
-    def __serialize_message_response(self, response):
-        return {
-            "success": response.result.success,
-            "created_bodies_monikers": response.result.created_bodies_monikers,
-            "modified_bodies_monikers": response.result.modified_bodies_monikers,
-        }

src/ansys/geometry/core/_grpc/_services/v1/conversions.py

@@ -44,6 +44,7 @@
     CurveGeometry as GRPCCurveGeometry,
     DrivingDimensionEntity as GRPCDrivingDimension,
     EdgeTessellation as GRPCEdgeTessellation,
+    EnhancedRepairToolMessage as GRPCEnhancedRepairToolResponse,
     Geometries as GRPCGeometries,
     Knot as GRPCKnot,
     MaterialEntity as GRPCMaterial,
@@ -1349,6 +1350,38 @@ def from_angle_to_grpc_quantity(input: "Measurement") -> GRPCQuantity:
     return GRPCQuantity(value_in_geometry_units=input.value.m_as(DEFAULT_UNITS.SERVER_ANGLE))
 
 
+def from_area_to_grpc_quantity(input: "Measurement") -> GRPCQuantity:
+    """Convert a ``Measurement`` containing an area to a gRPC quantity.
+
+    Parameters
+    ----------
+    input : Measurement
+        Source measurement data.
+
+    Returns
+    -------
+    GRPCQuantity
+        Converted gRPC quantity.
+    """
+    return GRPCQuantity(value_in_geometry_units=input.value.m_as(DEFAULT_UNITS.SERVER_AREA))
+
+
+def from_volume_to_grpc_quantity(input: "Measurement") -> GRPCQuantity:
+    """Convert a ``Measurement`` containing a volume to a gRPC quantity.
+
+    Parameters
+    ----------
+    input : Measurement
+        Source measurement data.
+
+    Returns
+    -------
+    GRPCQuantity
+        Converted gRPC quantity.
+    """
+    return GRPCQuantity(value_in_geometry_units=input.value.m_as(DEFAULT_UNITS.SERVER_VOLUME))
+
+
 def _nurbs_curves_compatibility(backend_version: "semver.Version", grpc_geometries: GRPCGeometries):
     """Check if the backend version is compatible with NURBS curves in sketches.
 
@@ -1568,3 +1601,84 @@ def get_tracker_response_with_created_bodies(response) -> dict:
         "created_bodies", []
     )
     return serialized_response
+
+
+def serialize_repair_command_response(response: GRPCEnhancedRepairToolResponse) -> dict:
+    """Serialize a EnhancedRepairToolResponse object into a dictionary.
+
+    Parameters
+    ----------
+    response : GRPCEnhancedRepairToolResponse
+        The gRPC EnhancedRepairToolResponse object to serialize.
+        A dictionary representation of the EnhancedRepairToolResponse object.
+    """
+    return {
+        "success": response.tracked_changes.command_response.success,
+        "found": response.found,
+        "repaired": response.repaired,
+        "complete_command_response": serialize_tracked_command_response(response.tracked_changes),
+        "created_bodies_monikers": [
+            created_body.id for created_body in response.tracked_changes.get("created_bodies", [])
+        ],
+        "modified_bodies_monikers": [
+            modified_body.id
+            for modified_body in response.tracked_changes.get("modified_bodies", [])
+        ],
+        "deleted_bodies_monikers": [
+            deleted_body.id for deleted_body in response.tracked_changes.get("deleted_bodies", [])
+        ],
+    }
+
+
+def response_problem_area_for_body(response) -> dict:
+    """Get a dictionary response from problem areas for a body.
+
+    Parameters
+    ----------
+    response
+        The response to convert.
+
+    Returns
+    -------
+    dict
+        A dictionary representation of the ProblemAreaForBody object.
+    """
+    return {
+        "problems": [{"id": res.problem_area_id, "bodies": res.body_ids} for res in response.result]
+    }
+
+
+def response_problem_area_for_face(response) -> dict:
+    """Get a dictionary response from problem areas for a face.
+
+    Parameters
+    ----------
+    response
+        The response to convert.
+
+    Returns
+    -------
+    dict
+        A dictionary representation of the ProblemAreaForFace object.
+    """
+    return {
+        "problems": [{"id": res.problem_area_id, "faces": res.face_ids} for res in response.result]
+    }
+
+
+def response_problem_area_for_edge(response) -> dict:
+    """Get a dictionary response from problem areas for an edge.
+
+    Parameters
+    ----------
+    response
+        The response to convert.
+
+    Returns
+    -------
+    dict
+        A dictionary representation of the ProblemAreaForEdge object.
+    """
+    return {
+        "problems": [{"id": res.problem_area_id, "edges": res.edge_ids} for res in response.result]
+    }