Skip to content

Commit d69ad38

Browse files
authored
Merge branch 'main' into pr-add_param_api_unittest
2 parents 64ffbd8 + d56a8b9 commit d69ad38

File tree

2 files changed

+17
-10
lines changed

2 files changed

+17
-10
lines changed

moveit_core/robot_model/src/joint_model_group.cpp

+8
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,14 @@ JointModelGroup::JointModelGroup(const std::string& group_name, const srdf::Mode
229229
{
230230
link_model_map_[link_model->getName()] = link_model;
231231
link_model_name_vector_.push_back(link_model->getName());
232+
// if this is the first link of the group with a valid parent and includes geometry (for example `base_link`) it should included
233+
if (link_model_with_geometry_vector_.empty() && link_model->getParentLinkModel() &&
234+
!link_model->getParentLinkModel()->getShapes().empty())
235+
{
236+
link_model_with_geometry_vector_.push_back(link_model->getParentLinkModel());
237+
link_model_with_geometry_name_vector_.push_back(link_model->getParentLinkModel()->getName());
238+
}
239+
// all child links with collision geometry should also be included
232240
if (!link_model->getShapes().empty())
233241
{
234242
link_model_with_geometry_vector_.push_back(link_model);

moveit_ros/moveit_servo/src/servo_node.cpp

+9-10
Original file line numberDiff line numberDiff line change
@@ -75,19 +75,18 @@ ServoNode::ServoNode(const rclcpp::NodeOptions& options)
7575
"in the launch file");
7676
}
7777

78-
// Check if a realtime kernel is available
79-
if (realtime_tools::has_realtime_kernel())
78+
// Configure SCHED_FIFO and priority
79+
if (realtime_tools::configure_sched_fifo(servo_params_.thread_priority))
8080
{
81-
if (realtime_tools::configure_sched_fifo(servo_params_.thread_priority))
82-
{
83-
RCLCPP_INFO_STREAM(node_->get_logger(), "Realtime kernel available, higher thread priority has been set.");
84-
}
85-
else
86-
{
87-
RCLCPP_WARN_STREAM(node_->get_logger(), "Could not enable FIFO RT scheduling policy.");
88-
}
81+
RCLCPP_INFO_STREAM(node_->get_logger(), "Enabled SCHED_FIFO and higher thread priority.");
8982
}
9083
else
84+
{
85+
RCLCPP_WARN_STREAM(node_->get_logger(), "Could not enable FIFO RT scheduling policy. Continuing with the default.");
86+
}
87+
88+
// Check if a realtime kernel is available
89+
if (!realtime_tools::has_realtime_kernel())
9190
{
9291
RCLCPP_WARN_STREAM(node_->get_logger(), "Realtime kernel is recommended for better performance.");
9392
}

0 commit comments

Comments
 (0)