Skip to content

Commit b960bed

Browse files
committed
[TODO] changes to make rviz_python_bindings working without the need to modify /rviz_common/visualization_frame. To be improved later
1 parent f8bd18c commit b960bed

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

rviz_python_bindings/include/rviz_python_bindings/visualizer_frame_py.hpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,7 @@ class VisualizerFramePy : public VisualizationFrame
1717
explicit VisualizerFramePy(
1818
QWidget * parent = nullptr);
1919

20-
~VisualizerFramePy()
21-
{
22-
ros_client_abstraction_->shutdown();
23-
}
20+
~VisualizerFramePy();
2421

2522
bool node_ok();
2623

rviz_python_bindings/src/visualizer_frame_py.cpp

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,15 @@
33
using namespace rviz_common;
44

55
VisualizerFramePy::VisualizerFramePy(QWidget * parent)
6-
: VisualizationFrame(parent)
6+
: VisualizationFrame(rviz_common::ros_integration::RosNodeAbstractionIface::WeakPtr(), parent)
77
{
88
ros_client_abstraction_ = std::make_unique<rviz_common::ros_integration::RosClientAbstraction>();
9-
this->rviz_ros_node_ = ros_client_abstraction_->init(0, nullptr, "rviz", false);
9+
rviz_ros_node_ = ros_client_abstraction_->init(0, nullptr, "rviz", false);
10+
}
11+
12+
VisualizerFramePy::~VisualizerFramePy()
13+
{
14+
ros_client_abstraction_->shutdown();
1015
}
1116

1217
bool VisualizerFramePy::node_ok()

0 commit comments

Comments
 (0)