stomp_ros
stomp_ros copied to clipboard
Check seed for validity
StompSmoothingAdapter:
Currently validity (ie collision free) is only checked after the first iteration. This means that even though a valid seed is given, STOMP might invalidate this during it's first iteration and never recover. This has led to discussions about an additional option allowing the adapter to fail and return the seed as a fall-back solution.
I propose to
- calculate the cost of the seed and thereby see if it's valid
- discuss about whether
num_iteration_after_valid: 0
should then be ignored and still at least one iteration performed - add an option to fail if the cost of the trajectory did not improve, returning the seed iff valid otherwise, fail if it isn't
this is a continuation of the discussion in #7 and #20
@simonschmeisser I'm on board with making stop evaluate the cost of the seed trajectory prior to the first iteration, it should be a matter of tweaking the main loop of stomp here