search

moveit / moveit2

:robot: MoveIt for ROS 2
https://moveit.ai/
BSD 3-Clause "New" or "Revised" License
989 stars 494 forks source link

Enable Cartesian Interpolations tests #2099

Open Abishalini opened 1 year ago

Abishalini commented 1 year ago

Is your feature request related to a problem? Please describe. During a sync with MoveIt, it was discovered that kinematics plugins could not be found in the tests added in moveit_core/robot_state/test/test_cartesian_interpolator.cpp and therefore some of the tests are temporarily commented out.

Describe the solution you'd like According to the dicussion here, the tests should be moved to the IK plugin package. Also, we should test computeCartesianPath() with a mock implementation of an IK solver so that we are not introducing solver-specific issues into our unit tests.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Sync PR - https://github.com/ros-planning/moveit2/pull/1900

github-actions[bot] commented 1 year ago

This issue is being labeled as stale because it has been open 45 days with no activity. It will be automatically closed after another 45 days without follow-ups.

github-actions[bot] commented 11 months ago

This issue was closed because it has been stalled for 45 days with no activity.