Skip to content

Commit dad01a0

Browse files
committed
[bugfix] fix pybind11 holder to prevent segfault on destruction
1 parent f2cb8ef commit dad01a0

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

python/pyxbot2_interface.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -145,9 +145,9 @@ PYBIND11_MODULE(pyxbot2_interface, m) {
145145
.def("difference",
146146
py::overload_cast<VecConstRef, VecConstRef>(&XBotInterface::difference, py::const_))
147147
.def("getForceTorque",
148-
py::overload_cast<>(&XBotInterface::getForceTorque, py::const_), rvp::reference_internal)
148+
py::overload_cast<>(&XBotInterface::getForceTorque, py::const_), rvp::reference)
149149
.def("getImu",
150-
py::overload_cast<>(&XBotInterface::getImu, py::const_), rvp::reference_internal)
150+
py::overload_cast<>(&XBotInterface::getImu, py::const_), rvp::reference)
151151
.def("getPose",
152152
py::overload_cast<string_const_ref>(&XBotInterface::getPose, py::const_))
153153
.def("getPose",
@@ -382,7 +382,7 @@ PYBIND11_MODULE(pyxbot2_interface, m) {
382382

383383

384384

385-
py::class_<Sensor>(m, "Sensor")
385+
py::class_<Sensor, Sensor::Ptr>(m, "Sensor")
386386
.def("getName",
387387
&Sensor::getName)
388388
.def("getTimestamp",
@@ -391,7 +391,7 @@ PYBIND11_MODULE(pyxbot2_interface, m) {
391391
&Sensor::isUpdated)
392392
;
393393

394-
py::class_<ImuSensor, Sensor>(m, "ImuSensor")
394+
py::class_<ImuSensor, Sensor, ImuSensor::Ptr>(m, "ImuSensor")
395395
.def("getAngularVelocity",
396396
py::overload_cast<>(&ImuSensor::getAngularVelocity, py::const_))
397397
.def("getLinearAcceleration",
@@ -403,7 +403,7 @@ PYBIND11_MODULE(pyxbot2_interface, m) {
403403
})
404404
;
405405

406-
py::class_<ForceTorqueSensor, Sensor>(m, "ForceTorqueSensor")
406+
py::class_<ForceTorqueSensor, Sensor, ForceTorqueSensor::Ptr>(m, "ForceTorqueSensor")
407407
.def("getWrench",
408408
py::overload_cast<>(&ForceTorqueSensor::getWrench, py::const_))
409409
;

0 commit comments

Comments
 (0)