session-py 0.166.0__tar.gz → 0.167.0__tar.gz

{session_py-0.166.0 → session_py-0.167.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: session_py
-Version: 0.166.0
+Version: 0.167.0
 Summary: A minimal Python geometry library
 Project-URL: Homepage, https://github.com/petrasvestartas/session_py
 Project-URL: Repository, https://github.com/petrasvestartas/session_py

{session_py-0.166.0 → session_py-0.167.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "session_py"
-version = "0.166.0"
+version = "0.167.0"
 description = "A minimal Python geometry library"
 authors = [{name = "Petras Vestartas", email = "petrasvestartas@gmail.com"}]
 license = {text = "MIT"}

{session_py-0.166.0 → session_py-0.167.0}/src/session_py/session.py

@@ -343,6 +343,11 @@ class Session:
 
     # ═══════════════════════════════════════════════════════════════════════════
     # Details - Add objects
+    #
+    # Every add_* below SKIPS an object that is None or carries nothing to draw, and returns
+    # None instead of a node: an empty point cloud, a polyline of fewer than two points, a
+    # mesh without faces. The check lives here so no caller has to write it, and so a scene
+    # never holds an object a viewer cannot render.
     # ═══════════════════════════════════════════════════════════════════════════
 
     def _add_object(self, collection, obj, type_prefix, parent=None):
@@ -372,37 +377,61 @@ class Session:
             + [e.guid for e in self.objects.elements]
         )
 
-    def add_point(self, point: Point, parent: TreeNode | None = None) -> TreeNode:
+    def add_point(self, point: Point, parent: TreeNode | None = None) -> TreeNode | None:
+        if point is None:
+            return None
         return self._add_object(self.objects.points, point, "point", parent)
 
-    def add_line(self, line: "Line", parent: TreeNode | None = None) -> TreeNode:
+    def add_line(self, line: "Line", parent: TreeNode | None = None) -> TreeNode | None:
+        if line is None:
+            return None
         return self._add_object(self.objects.lines, line, "line", parent)
 
-    def add_plane(self, plane: "Plane", parent: TreeNode | None = None) -> TreeNode:
+    def add_plane(self, plane: "Plane", parent: TreeNode | None = None) -> TreeNode | None:
+        if plane is None:
+            return None
         return self._add_object(self.objects.planes, plane, "plane", parent)
 
-    def add_obb(self, bbox: "OBB", parent: TreeNode | None = None) -> TreeNode:
+    def add_obb(self, bbox: "OBB", parent: TreeNode | None = None) -> TreeNode | None:
+        if bbox is None:
+            return None
         return self._add_object(self.objects.bboxes, bbox, "bbox", parent)
 
-    def add_polyline(self, polyline: "Polyline", parent: TreeNode | None = None) -> TreeNode:
+    def add_polyline(self, polyline: "Polyline", parent: TreeNode | None = None) -> TreeNode | None:
+        if polyline is None or polyline.point_count() < 2:
+            return None
         return self._add_object(self.objects.polylines, polyline, "polyline", parent)
 
-    def add_pointcloud(self, pointcloud: "PointCloud", parent: TreeNode | None = None) -> TreeNode:
+    def add_pointcloud(self, pointcloud: "PointCloud", parent: TreeNode | None = None) -> TreeNode | None:
+        if pointcloud is None or pointcloud.is_empty():
+            return None
         return self._add_object(self.objects.pointclouds, pointcloud, "pointcloud", parent)
 
-    def add_mesh(self, mesh: "Mesh", parent: TreeNode | None = None) -> TreeNode:
+    def add_mesh(self, mesh: "Mesh", parent: TreeNode | None = None) -> TreeNode | None:
+        if mesh is None or mesh.is_empty() or mesh.number_of_faces() == 0:
+            return None
         return self._add_object(self.objects.meshes, mesh, "mesh", parent)
 
-    def add_nurbscurve(self, nurbscurve: "NurbsCurve", parent: TreeNode | None = None) -> TreeNode:
+    def add_nurbscurve(self, nurbscurve: "NurbsCurve", parent: TreeNode | None = None) -> TreeNode | None:
+        if nurbscurve is None or nurbscurve.cv_count() < 2:
+            return None
         return self._add_object(self.objects.nurbscurves, nurbscurve, "nurbscurve", parent)
 
-    def add_nurbssurface(self, nurbssurface: "NurbsSurface", parent: TreeNode | None = None) -> TreeNode:
+    def add_nurbssurface(self, nurbssurface: "NurbsSurface", parent: TreeNode | None = None) -> TreeNode | None:
+        if nurbssurface is None or nurbssurface.cv_count() == 0:
+            return None
         return self._add_object(self.objects.nurbssurfaces, nurbssurface, "nurbssurface", parent)
 
-    def add_brep(self, brep: "BRep", parent: TreeNode | None = None) -> TreeNode:
+    def add_brep(self, brep: "BRep", parent: TreeNode | None = None) -> TreeNode | None:
+        if brep is None or (brep.face_count() == 0 and brep.vertex_count() == 0):
+            return None
         return self._add_object(self.objects.breps, brep, "brep", parent)
 
-    def add_element(self, element: "Element", parent: TreeNode | None = None) -> TreeNode:
+    def add_element(self, element: "Element", parent: TreeNode | None = None) -> TreeNode | None:
+        # Kept even with no geometry: an Element is a data record - features, insertion vectors,
+        # element_data a consumer reads back - and dropping one would lose that on a round trip.
+        if element is None:
+            return None
         return self._add_object(self.objects.elements, element, "element", parent)
 
     def add_component(self, component: Any, parent: TreeNode | None = None) -> TreeNode:
@@ -432,10 +461,13 @@ class Session:
         Parameters
         ----------
         node : TreeNode
-            The TreeNode to add.
+            The TreeNode to add. None is ignored, so passing an add_* result straight
+            through works even when that add_* skipped its geometry.
         parent : TreeNode, optional
             Parent TreeNode (defaults to root if not provided).
         """
+        if node is None:
+            return
         if parent is None:
             self.tree.add(node, self.tree.root)
         else:

{session_py-0.166.0 → session_py-0.167.0}/src/session_py/session_test.py

@@ -215,6 +215,42 @@ def test_session_add_element():
     MINI_CHECK(session.graph.has_node(plate.guid))
 
 
+@MINI_TEST("Session", "Add Empty Geometry")
+def test_session_add_empty_geometry():
+    from session_py import Session
+    from session_py import Point
+    from session_py import Polyline
+    from session_py import PointCloud
+    from session_py import Mesh
+    from session_py import NurbsCurve
+    from session_py import NurbsSurface
+    from session_py import BRep
+
+    # Nothing to draw is never added: the caller does not test its geometry first.
+    session = Session()
+    group = session.add_group("empty")
+
+    MINI_CHECK(session.add_point(None, group) is None)
+    MINI_CHECK(session.add_polyline(Polyline([Point(0, 0, 0)]), group) is None)
+    MINI_CHECK(session.add_pointcloud(PointCloud([], [], []), group) is None)
+    MINI_CHECK(session.add_mesh(Mesh(), group) is None)
+    MINI_CHECK(session.add_nurbscurve(NurbsCurve(), group) is None)
+    MINI_CHECK(session.add_nurbssurface(NurbsSurface(), group) is None)
+    MINI_CHECK(session.add_brep(BRep(), group) is None)
+
+    # A mesh with vertices but no faces draws nothing either.
+    vertices_only = Mesh()
+    vertices_only.add_vertex(Point(0, 0, 0), 0)
+    MINI_CHECK(session.add_mesh(vertices_only, group) is None)
+
+    # add(add_XXX(...), group) stays a valid one-liner when the geometry was skipped.
+    session.add(session.add_mesh(Mesh(), group), group)
+
+    MINI_CHECK(len(session.lookup) == 0)
+    MINI_CHECK(len(session.order()) == 0)
+    MINI_CHECK(len(group.children) == 0)
+
+
 @MINI_TEST("Session", "Add Group")
 def test_session_add_group():
     from session_py import Session