Skip to content

Commit 0fa79d6

Browse files
author
Marcus Liebhardt
authored
Merge pull request #118 from yujinrobot/enableCB_fix
[diff_drive_pose_controller] Change target pose even if controller al…
2 parents 38d4517 + ce834cb commit 0fa79d6

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

yocs_diff_drive_pose_controller/src/diff_drive_pose_controller_ros.cpp

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,14 +221,18 @@ void DiffDrivePoseControllerROS::controlMaxVelCB(const std_msgs::Float32ConstPtr
221221

222222
void DiffDrivePoseControllerROS::enableCB(const std_msgs::StringConstPtr msg)
223223
{
224-
if (this->enable())
224+
if (!msg->data.empty())
225225
{
226226
goal_frame_name_ = msg->data;
227-
ROS_INFO_STREAM("Controller has been enabled. [" << name_ << "] with goal frame [" << goal_frame_name_ << "]");
227+
}
228+
229+
if (this->enable())
230+
{
231+
ROS_INFO_STREAM("Controller has been enabled [" << name_ << "] with goal frame [" << goal_frame_name_ << "].");
228232
}
229233
else
230234
{
231-
ROS_INFO_STREAM("Controller was already enabled. [" << name_ <<"] with Goal frame [" << goal_frame_name_ << "]");
235+
ROS_INFO_STREAM("Controller was already enabled [" << name_ <<"], now tracking goal frame [" << goal_frame_name_ << "].");
232236
}
233237
}
234238

0 commit comments

Comments
 (0)