Fix for issue #65.

The mini-tree builder now accepts configurations with `min_leaf_size > 1`.
madmann91 · Oct 2, 2023 · 02ee929 · 02ee929
1 parent ad3d436
commit 02ee929
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/bvh/v2/mini_tree_builder.h b/src/bvh/v2/mini_tree_builder.h
@@ -256,7 +256,7 @@ class MiniTreeBuilder {
         }
 
         typename SweepSahBuilder<Node>::Config config = config_;
-        config.max_leaf_size = 1; // Needs to have only one mini-tree in each leaf
+        config.max_leaf_size = config.min_leaf_size = 1; // Needs to have only one mini-tree in each leaf
         auto bvh = SweepSahBuilder<Node>::build(bboxes, centers, config);
 
         // Compute the offsets to apply to primitive and node indices

diff --git a/src/bvh/v2/top_down_sah_builder.h b/src/bvh/v2/top_down_sah_builder.h
@@ -61,6 +61,7 @@ class TopDownSahBuilder {
         , config_(config)
     {
         assert(bboxes.size() == centers.size());
+        assert(config.min_leaf_size <= config.max_leaf_size);
     }
 
     virtual std::vector<size_t>& get_prim_ids() = 0;