removed OCC specific properties from interface

Author: chenkasirer

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## Unreleased
 
 ### Added
+
 * Implemented `to_points` method in `compas.datastructures.Mesh`, which before raised a `NotImplementedError`.
 * Implemented `compute_aabb` method in `compas.datastructures.Datastructure`, which before raised a `NotImplementedError`. Made use of the `compas.geometry.bbox.bounding_box` function.
 * Implemented `compute_obb` method in `compas.datastructures.Datastructure`, which before raised a `NotImplementedError`. Made use of the `compas.geometry.bbox_numpy.oriented_bounding_box_numpy` function.
@@ -20,8 +21,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Added missing property `centroid` in `compas_rhino.geometry.RhinoBrep`.
 * Added missing property `curves` in `compas_rhino.geometry.RhinoBrep`.
 * Added missing property `is_closed` in `compas_rhino.geometry.RhinoBrep`.
-* Added missing property `is_compound` in `compas_rhino.geometry.RhinoBrep`.
-* Added missing property `is_compoundsolid` in `compas_rhino.geometry.RhinoBrep`.
 * Added missing property `is_orientable` in `compas_rhino.geometry.RhinoBrep`.
 * Added missing property `is_surface` in `compas_rhino.geometry.RhinoBrep`.
 * Added missing property `is_valid` in `compas_rhino.geometry.RhinoBrep`.
@@ -42,6 +41,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Removed
 
+* Removed property `is_compound` from `compas.geometry.Brep` as OCC specific.
+* Removed property `is_compoundsolid` from `compas.geometry.Brep` as OCC specific.
+* Removed property `solids` from `compas.geometry.Brep` as OCC specific.
+* Removed property `shells` from `compas.geometry.Brep` as OCC specific.
 
 ## [2.13.0] 2025-06-04
 

src/compas/geometry/brep/brep.py

@@ -177,30 +177,10 @@ def native_brep(self):
     def orientation(self):
         raise NotImplementedError
 
-    @property
-    def type(self):
-        raise NotImplementedError
-
     @property
     def is_valid(self):
         raise NotImplementedError
 
-    @property
-    def is_shell(self):
-        raise NotImplementedError
-
-    @property
-    def is_solid(self):
-        raise NotImplementedError
-
-    @property
-    def is_compound(self):
-        raise NotImplementedError
-
-    @property
-    def is_compoundsolid(self):
-        raise NotImplementedError
-
     @property
     def is_orientable(self):
         raise NotImplementedError

src/compas_rhino/geometry/brep/brep.py

@@ -236,16 +236,6 @@ def is_closed(self):
         assert self._brep
         return self._brep.IsSolid
 
-    @property
-    def is_compound(self):
-        # TODO: clarify. according to the internets compound brep is actually a container for several breps, not sure that's possible with a Rhino Brep.
-        return False
-
-    @property
-    def is_compoundsolid(self):
-        # TODO: see above
-        return False
-
     @property
     def is_convex(self):
         raise NotImplementedError("Convexity check is not implemented for Rhino Breps.")
@@ -281,26 +271,11 @@ def orientation(self):
         # TODO: align this with compas.geometry.BrepOrientation
         return self._brep.SolidOrientation
 
-    @property
-    def shells(self):
-        # TODO: can create shell from brep but have to specify which faces to eliminate in order to hollow out the brep, doesn't seem like the intention.
-        # TODO: is this about traversing a compound brep?
-        raise NotImplementedError("Shells are not implemented for Rhino Breps.")
-
-    @property
-    def solids(self):
-        # TODO: same as above
-        raise NotImplementedError("Solids are not implemented for Rhino Breps.")
-
     @property
     def surfaces(self):
         assert self._brep
         return [[RhinoNurbsSurface.from_native(s.ToNurbsSurface()) for s in self._brep.Surfaces]]
 
-    @property
-    def type(self):
-        # TODO: seems like an OCC specific thinh, rename to occ_type and remove from interface?
-        raise NotImplementedError("Type is not implemented for Rhino Breps.")
 
     # ==============================================================================
     # Constructors