-
Notifications
You must be signed in to change notification settings - Fork 937
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
Split collision proximity threshold #2008
Split collision proximity threshold #2008
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good in general, I'll stress test this a bit. Please rebase and then lgtm after #1982
moveit_experimental/moveit_jog_arm/include/moveit_jog_arm/jog_calcs.h
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/config/ur_simulated_config.yaml
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
Did you test it? Maybe on the UR5 simulation? I can do that, if you want |
a5c0bad
to
dce6344
Compare
I did test it with the UR5 simulation on our current project. |
Looks good to me! |
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
moveit_experimental/moveit_jog_arm/src/collision_check_thread.cpp
Outdated
Show resolved
Hide resolved
Looks good in general, Travis CI seems to require another rebase. Please consider the alternative implementation I suggested. |
3aaaa5f
to
357052e
Compare
619fdc7
to
91b989e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Could you take care about the last comment I have? Then lgtm
Congrats on getting your first MoveIt pull request merged and improving open source robotics! |
* separate proximity threshold values for self-collisions and scene collisions * increase default value of scene collision proximity threshold * deprecate old parameters
* separate proximity threshold values for self-collisions and scene collisions * increase default value of scene collision proximity threshold * deprecate old parameters
* separate proximity threshold values for self-collisions and scene collisions * increase default value of scene collision proximity threshold * deprecate old parameters
Co-authored-by: Henning Kayser <henningkayser@picknik.ai> Co-authored-by: Tyler Weaver <maybe@tylerjw.dev>
Description
Replaces
collision_proximity_threshold
config parameter withself_collision_proximity_threshoold
andscene_collision_proximity_threshold
, so that a lower self-collision threshold can be used. This helps handle near-collisions we suspect are from links that are always near to each other, although unlikely to collide.Checklist