Fix default broad phase in python

Author: zfergus

python/src/candidates/candidates.cpp

@@ -1,7 +1,6 @@
 #include <common.hpp>
 
 #include <ipc/candidates/candidates.hpp>
-#include <ipc/broad_phase/broad_phase.hpp>
 
 namespace py = pybind11;
 using namespace ipc;
@@ -25,7 +24,8 @@ void define_candidates(py::module_& m)
                 broad_phase: Broad phase to use.
             )ipc_Qu8mg5v7",
             py::arg("mesh"), py::arg("vertices"),
-            py::arg("inflation_radius") = 0, py::arg("broad_phase") = nullptr)
+            py::arg("inflation_radius") = 0,
+            py::arg("broad_phase") = make_default_broad_phase())
         .def(
             "build",
             py::overload_cast<
@@ -46,7 +46,8 @@ void define_candidates(py::module_& m)
                 broad_phase: Broad phase to use.
             )ipc_Qu8mg5v7",
             py::arg("mesh"), py::arg("vertices_t0"), py::arg("vertices_t1"),
-            py::arg("inflation_radius") = 0, py::arg("broad_phase") = nullptr)
+            py::arg("inflation_radius") = 0,
+            py::arg("broad_phase") = make_default_broad_phase())
         .def("__len__", &Candidates::size)
         .def("empty", &Candidates::empty)
         .def("clear", &Candidates::clear)
@@ -127,7 +128,7 @@ void define_candidates(py::module_& m)
             )ipc_Qu8mg5v7",
             py::arg("mesh"), py::arg("vertices_t0"), py::arg("vertices_t1"),
             py::arg("dhat"), py::arg("min_distance") = 0.0,
-            py::arg("broad_phase") = nullptr,
+            py::arg("broad_phase") = make_default_broad_phase(),
             py::arg("narrow_phase_ccd") = DEFAULT_NARROW_PHASE_CCD)
         .def(
             "save_obj", &Candidates::save_obj, py::arg("filename"),

python/src/collisions/normal/normal_collisions.cpp

@@ -1,7 +1,6 @@
 #include <common.hpp>
 
 #include <ipc/collisions/normal/normal_collisions.hpp>
-#include <ipc/broad_phase/broad_phase.hpp>
 
 namespace py = pybind11;
 using namespace ipc;
@@ -27,7 +26,8 @@ void define_normal_collisions(py::module_& m)
                 broad_phase: Broad-phase to use.
             )ipc_Qu8mg5v7",
             py::arg("mesh"), py::arg("vertices"), py::arg("dhat"),
-            py::arg("dmin") = 0, py::arg("broad_phase") = nullptr)
+            py::arg("dmin") = 0,
+            py::arg("broad_phase") = make_default_broad_phase())
         .def(
             "build",
             py::overload_cast<

python/src/ipc.cpp

@@ -1,7 +1,6 @@
 #include <common.hpp>
 
 #include <ipc/ipc.hpp>
-#include <ipc/broad_phase/broad_phase.hpp>
 #include <ipc/config.hpp>
 
 #include <igl/edges.h>
@@ -33,7 +32,8 @@ void define_ipc(py::module_& m)
             True if <b>any</b> collisions occur.
         )ipc_Qu8mg5v7",
         py::arg("mesh"), py::arg("vertices_t0"), py::arg("vertices_t1"),
-        py::arg("min_distance") = 0.0, py::arg("broad_phase") = nullptr,
+        py::arg("min_distance") = 0.0,
+        py::arg("broad_phase") = make_default_broad_phase(),
         py::arg("narrow_phase_ccd") = DEFAULT_NARROW_PHASE_CCD);
 
     m.def(
@@ -56,7 +56,8 @@ void define_ipc(py::module_& m)
             A step-size :math:`\in [0, 1]` that is collision free. A value of 1.0 if a full step and 0.0 is no step.
         )ipc_Qu8mg5v7",
         py::arg("mesh"), py::arg("vertices_t0"), py::arg("vertices_t1"),
-        py::arg("min_distance") = 0.0, py::arg("broad_phase") = nullptr,
+        py::arg("min_distance") = 0.0,
+        py::arg("broad_phase") = make_default_broad_phase(),
         py::arg("narrow_phase_ccd") = DEFAULT_NARROW_PHASE_CCD);
 
     m.def(
@@ -72,7 +73,8 @@ void define_ipc(py::module_& m)
         Returns:
             A boolean for if the mesh has intersections.
         )ipc_Qu8mg5v7",
-        py::arg("mesh"), py::arg("vertices"), py::arg("broad_phase") = nullptr);
+        py::arg("mesh"), py::arg("vertices"),
+        py::arg("broad_phase") = make_default_broad_phase());
 
     m.def(
         "edges",