Skip to content

Commit

Permalink
ComponentManager: switch off parameter services and event publisher
Browse files Browse the repository at this point in the history
So far, the component manager has no parameters anyway, so no need for
services to configure them or topics to publish events.

An attempt to limit the number of services / topics.
(https://discourse.ros.org/t/ros2-default-behavior-wifi/13460/35)

Signed-off-by: Martijn Buijs <martijn.buijs@gmail.com>
  • Loading branch information
mbuijs committed Sep 25, 2020
1 parent eb9fc5f commit b069705
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion rclcpp_components/src/component_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,9 @@ ComponentManager::ComponentManager(
std::weak_ptr<rclcpp::Executor> executor,
std::string node_name,
const rclcpp::NodeOptions & node_options)
: Node(std::move(node_name), node_options),
: Node(std::move(node_name), rclcpp::NodeOptions(node_options)
.start_parameter_services(false)
.start_parameter_event_publisher(false)),
executor_(executor)
{
loadNode_srv_ = create_service<LoadNode>(
Expand Down

0 comments on commit b069705

Please sign in to comment.