moveit / moveit_visual_tools

Helper functions for displaying and debugging MoveIt! data in Rviz via published markers
BSD 3-Clause "New" or "Revised" License
151 stars 102 forks source link

Use first trajectory point as start state for visualization #49

Closed henningkayser closed 5 years ago

henningkayser commented 5 years ago

When visualizing trajectories of subgroups the previous implementation would use the shared initial robot state for populating the joint values outside the group. This can have strange effects like apparent link collisions even though the original trajectory is collision free. If we have a robot trajectory with fully populated robot states we should use these instead.

bmagyar commented 5 years ago

I've restarted the CI jobs, 3/3 "succeeded", 2/3 failed on a post-build timeout :-/

davetcoleman commented 5 years ago

Travis error:

moveit_ros/perception/mesh_filter/test/mesh_filter_test.cpp

davetcoleman commented 5 years ago

@henningkayser can you compare .travis.yml from /moveit repo to this repo? maybe there are some changes we need to bring in to fix travis?

rhaschke commented 5 years ago

@henningkayser can you compare .travis.yml from /moveit repo to this repo?

@davetcoleman, there is an open PR (#47) to address the Travis issues waiting for your approval since a month or so.

davetcoleman commented 5 years ago

I had forgot about that PR, thanks Robert!

rhaschke commented 5 years ago

Closing & re-opening to trigger Travis rebuild.