yujin_ocs
yujin_ocs copied to clipboard
[yocs_diff_drive_controller] hard-coded factor for parameter
@AlexReimann I just ran across this:
[...]
dist_eps_ = dist_eps_ * 0.2;
[...]
orient_eps_ = orient_thres_ * 0.2
https://github.com/yujinrobot/yujin_ocs/blob/indigo-devel/yocs_diff_drive_pose_controller/src/diff_drive_pose_controller_ros.cpp#L113-L129
What is the reason for (was the motivation behind) using this hard-coded factor?
@stonier FYI
Was already there before I touched the code, question back to you ;)
Right, GitHub's blame was misleading. :smile:
This factor got introduced here: https://github.com/yujinrobot/yujin_ocs/commit/fb26d5a492e46b4b0a09b640596511226e31d8f8
Will think more about it and remove if I cannot come up with a good reason for it.