ros-planning/moveit_core — pull request #198 by hersh (2014-09-04 20:04:28+00:00)
Title: Removed problematic assertion from default_constraint_samplers.cpp

When doing a task where moveit is planning repeatedly, this assertion consistently fails for me after a few minutes of running.

Looking at the code, I see that the function it is in, callIK(), already returns a bool indicating success or failure, and callIK() is called from a sampling loop that keeps trying until success. Therefore I don't see a problem with changing the "assert" to "return".