Skip to content

Commit d8802bf

Browse files
committed
Added wrapping for logging
1 parent 003a3c5 commit d8802bf

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

nav2_costmap_2d/plugins/costmap_filters/binary_filter.cpp

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,20 @@ void BinaryFilter::initializeFilter(
8484
BinaryParameter param_struct;
8585

8686
declareParameter(param + "." + "node_name", rclcpp::PARAMETER_STRING);
87-
// This throws an error if param is not initialized
88-
param_struct.node_name =
89-
node->get_parameter(name_ + "." + param + "." + "node_name").as_string();
87+
try {
88+
param_struct.node_name =
89+
node->get_parameter(name_ + "." + param + "." + "node_name").as_string();
90+
} catch (rclcpp::exceptions::ParameterUninitializedException & ex) {
91+
throw std::runtime_error("Node name not defined for " + param);
92+
}
9093

9194
declareParameter(param + "." + "param_name", rclcpp::PARAMETER_STRING);
92-
param_struct.param_name =
93-
node->get_parameter(name_ + "." + param + "." + "param_name").as_string();
95+
try {
96+
param_struct.param_name =
97+
node->get_parameter(name_ + "." + param + "." + "param_name").as_string();
98+
} catch (rclcpp::exceptions::ParameterUninitializedException & ex) {
99+
throw std::runtime_error("Parameter name not defined for " + param);
100+
}
94101

95102
// Take default value from parameter server if not specified
96103
declareParameter(param + "." + "default_state", rclcpp::ParameterValue(default_state_));

0 commit comments

Comments
 (0)