openscad-cpp-evaluator 1.49.5__tar.gz → 1.49.6__tar.gz

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openscad_cpp_evaluator
-Version: 1.49.5
+Version: 1.49.6
 Summary: C++ OpenSCAD evaluator with Python bindings
 Author: NeuSCAD Contributors
 License-Expression: MIT AND OFL-1.1

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/include/openscad_cpp_evaluator/colored_body.hpp

@@ -132,6 +132,12 @@ struct ColoredBody {
     bool isDisplayOnly() const { return rawMesh.has_value(); }
 };
 
+// A `%` body or a `#` operand's ghost copy: drawn, never part of the model,
+// so an operation that consumes its children's geometry must skip it.
+inline bool isModifierGhost(const ColoredBody& b) {
+    return b.role == BodyRole::Background || b.role == BodyRole::HighlightGhost;
+}
+
 // The cached questions -- see ColoredBody::knownEmpty. Ask Manifold once per
 // body, never once per tree level.
 inline bool bodyIsEmpty(ColoredBody& b) {

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "scikit_build_core.build"
 
 [project]
 name = "openscad_cpp_evaluator"
-version = "1.49.5"
+version = "1.49.6"
 description = "C++ OpenSCAD evaluator with Python bindings"
 readme = "README.md"
 requires-python = ">=3.12"

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/src/builtins/booleans.cpp

@@ -64,11 +64,14 @@ RoleSplit splitByRole(const std::vector<ColoredBody>& bodies) {
 
 // Unions every 2D child into a single CrossSection; nullopt if there are no
 // 2D children at all. Mirrors _to_cross_section. Shared by
-// linear_extrude/rotate_extrude/roof (extrude.cpp, roof.cpp).
+// linear_extrude/rotate_extrude/offset/roof (extrude.cpp, roof.cpp).
+// A `%` body and a `#` operand's ghost copy are display aids, not model:
+// unioning them in filled a differenced-away notch back in (BelfrySCAD
+// #866). OpenSCAD shows neither under these operations, so they are dropped.
 std::optional<manifold::CrossSection> toCrossSection(const std::vector<ColoredBody>& bodies) {
     std::optional<manifold::CrossSection> cs;
     for (const ColoredBody& b : bodies) {
-        if (!b.section) continue;
+        if (!b.section || isModifierGhost(b)) continue;
         cs = cs ? (*cs + *b.section) : *b.section;
     }
     return cs;

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/src/builtins/extrude.cpp

@@ -471,7 +471,9 @@ std::vector<ColoredBody> generateProjection(Evaluator&, const CSGParams& params,
     const std::vector<ColoredBody> bodies = flattenCsgTree(children);
     std::vector<ColoredBody> bodies3d;
     for (const ColoredBody& c : bodies) {
-        if (c.body) bodies3d.push_back(c);
+        // Not a `%` body or a `#` ghost: projecting them drew the
+        // differenced-away part's outline (BelfrySCAD #866). See toCrossSection.
+        if (c.body && !isModifierGhost(c)) bodies3d.push_back(c);
     }
     if (bodies3d.empty()) return {};
 

{openscad_cpp_evaluator-1.49.5 → openscad_cpp_evaluator-1.49.6}/tests/test_extrude_roof.cpp

@@ -260,6 +260,31 @@ TEST(Projection, CutFillsWhereAPolyhedronOverlapsItself) {
     EXPECT_NEAR(e.bodies[0].section->Area(), 6.0, 1e-6);
 }
 
+// A `#` operand is drawn again as a ghost and a `%` body is drawn but not
+// modelled; neither may reach the operation. Both leaked into projection()
+// and every 2D-consuming operation (BelfrySCAD #866): the ghost refilled
+// the cut-away corner and the `%` cube added its square.
+TEST(Projection, ModifierGhostsAreNotProjected) {
+    for (const char* src : {
+             "projection() difference() { cube([4,4,1]); #translate([-1,-1,-1]) cube(3); }",
+             "projection() union() { difference() { cube([4,4,1]); #translate([-1,-1,-1]) cube(3); }"
+             "  %translate([10,0,0]) cube(1); }"}) {
+        Evaluated e = evalSrc(src);
+        double area = 0;
+        for (const ColoredBody& b : e.bodies)
+            if (b.section) area += b.section->Area();
+        EXPECT_NEAR(area, 12.0, 1e-6) << src;
+    }
+}
+
+TEST(LinearExtrude, ModifierGhostsAreNotExtruded) {
+    Evaluated e = evalSrc("linear_extrude(1) difference() { square(4); #translate([-1,-1]) square(3); }");
+    double volume = 0;
+    for (const ColoredBody& b : e.bodies)
+        if (b.body) volume += b.body->Volume();
+    EXPECT_NEAR(volume, 12.0, 1e-6);
+}
+
 // -- offset -----------------------------------------------------------------
 
 TEST(Offset, RoundGrowsAreaBySquarePerimeterPlusCircleCorners) {