Replace moveit_planners with moveit_planners_chomp#253
Conversation
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
Signed-off-by: Mabel Zhang <mabel@openrobotics.org>
|
Tested with the dual arm rviz demo and everything still runs fine. If this fixes CI I'll approve for you. It does make me wonder, though, why the other fix worked when moveit_planners was still included in the file. |
j-herman
left a comment
There was a problem hiding this comment.
Verified it all still works with the bimanual manipulation demo
|
I don't know what the CI is checking, but seems like it's doing more than our manual builds. |
Trying to fix CI broken in #239 and #250
See #250 (comment) and subsequent comments.
CI might be failing because
moveit_plannersdoesn't exist in the Docker image. These ones exist though:moveit_planners_ompl(already a dependency) andmoveit_planners_chomp.Replace
moveit_plannerswithmoveit_planners_chompto see if that fixes CI.Rearrange in alphabetical order while I'm at it.