-
Notifications
You must be signed in to change notification settings - Fork 317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Link3 and Link4 have self collision problem #267
Comments
If at all, this issue should be reported in panda_moveit_config. Which version of panda_moveit_config are you using? The latest noetic-devel branch from source? At least, I cannot confirm your issue with this latest version. |
thank you, @rhaschke, for your response.
As I mentioned in the beginning, this issue also existed when merging the PR.Have a look on this #199 (comment)
I can open an issue in that repo, but since I was not sure about it and I saw this #199 (comment) I opened it here. Let me know what do you think. |
The coarse collision models are OK and resemble what is used by the (real) robot controller. Thus, please open an issue in panda_moveit_config. Please also provide the joint configuration causing the collision. |
Hi @kouroshD thanks for catching this. I completely missed that in my original comment. However I agree with @rhaschke and cannot reproduce this with the latest In any case reducing the capsules to only 1cm length seems a bit aggressive. This won't protect the entity of the shells anymore: @rhaschke if this is actually an issue, isn't it required to also fix the URDF here or is |
I guess @kouroshD isn't using the latest MoveIt version, which introduced auto-disabling collisions in SRDF. The URDF is fine. |
@kouroshD, did this issue resolved when using the latest MoveIt version? If not, please file a bug report at panda_moveit_config and provide an exact joint configuration triggering the collision. |
Thank you for your support @rhaschke and @gollth . In any case, I will update you about the problem on this issue. |
No. ROS is targeting the system installation of python. I suggest to have a look here: https://github.com/RoboStack/ros-noetic |
This issue seems to be solved. I will close it now. Free free to reopen it if the issue is still there. |
Recently, this PR has been merged related to the detailed and coarse collision models.
At the end of PR, in this comment #199 (comment) as shown
link3
andlink4
have self collision problem with other links of the robot and in fact in RViz are shown in red.Here is the gif from that comment:
I could also reproduce this problem.
I have investigated a bit, and apparently the problem is from
franka_ros/franka_description/robots/panda_arm.xacro
Lines 55 to 59 in 54ad920
franka_ros/franka_description/robots/panda_arm.xacro
Lines 71 to 75 in 54ad920
Apparently, the values for
radius
andlength
are high.A workaround I found was to reduce both of those values to
0.01
, and the self collision problem was fixed, as you can see in the following gif:selfcollision-2022-07-07_21.13.56.mp4
However, I am not sure how those values are computed and set in the
xacro
files, and if the problem is from those values or from how they are used later on in the pipeline.cc @gollth and @rhaschke
The text was updated successfully, but these errors were encountered: