Merge pull request #1432 from compas-dev/brepface_frame_at

added frame_at to BrepFace

Author: chenkasirer

CHANGELOG.md

@@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+* Added method `frame_at` to `compas.geometry.BrepFace`,
+* Added method `frame_at` to `compas_rhino.geometry.RhinoBrepFace`,
+
 ### Changed
 
 * Fixed publish to YAK via CI workflow.

src/compas/geometry/brep/face.py

@@ -292,6 +292,24 @@ def add_loops(self, loops, *args, **kwargs):
         """
         raise NotImplementedError
 
+    def frame_at(self, u, v):
+        """Returns the frame at the given uv parameters.
+
+        Parameters
+        ----------
+        u : float
+            The u parameter.
+        v : float
+            The v parameter.
+
+        Returns
+        -------
+        :class:`compas.geometry.Frame`
+            The frame at the given uv parameters.
+
+        """
+        raise NotImplementedError
+
     def try_get_nurbssurface(
         self,
         precision,

src/compas_rhino/geometry/brep/face.py

@@ -283,3 +283,24 @@ def as_brep(self):
 
         """
         return Brep.from_native(self._face.ToBrep())
+
+    def frame_at(self, u, v):
+        """Returns the frame at the given uv parameters.
+
+        Parameters
+        ----------
+        u : float
+            The u parameter.
+        v : float
+            The v parameter.
+
+        Returns
+        -------
+        :class:`compas.geometry.Frame`
+            The frame at the given uv parameters.
+
+        """
+        success, rhino_plane = self._face.FrameAt(u, v)
+        if not success:
+            raise ValueError("Failed to get frame at uv parameters: ({},{}).".format(u, v))
+        return plane_to_compas_frame(rhino_plane)