search

moveit / moveit_resources

URDFs, meshes, and config packages for MoveIt testing
61 stars 115 forks source link

Recreate moveit_resources_panda_moveit_config via MSA #92

Closed rhaschke closed 2 years ago

rhaschke commented 3 years ago

... as suggested by @v4hn in https://github.com/ros-planning/moveit_resources/issues/89#issuecomment-871656557 Thus, moveit_resources_panda_moveit_config diverges from panda_moveit_config!

v4hn commented 3 years ago

@rhaschke As far as I know you proposed this PR as a way to solve #89, but your patch does not include the required broadcaster.

Did you use an older version of MSA to generate the config? Also I would propose to keep most of the additional files in this repository. Some, like test_environment.launch are useful helpers even if they are not (should not necessarily be) created by MSA automatically.

I filed #97 to solve the immediate issue for now.

rhaschke commented 3 years ago

@v4hn, thanks for pointing out the open issues. I ran MSA once again with a recent version of MoveIt. Probably, I used the latest release before, which is already rather old... I also successfully ran the tests of the main MoveIt repo.