personalrobotics / or_trajopt

OpenRAVE plugin to expose TrajOpt code as an OpenRAVE planner
BSD 3-Clause "New" or "Revised" License
2 stars 3 forks source link

TrajoptPlanner throws (Self)CollisionPlanningError and ConstraintViol… #15

Closed gilwoolee closed 8 years ago

gilwoolee commented 8 years ago

…ationPlanningError whenever possible.

psigen commented 8 years ago

Thanks Gilwoo! I added a few comments, but overall the new exception handling looks much more helpful.

gilwoolee commented 8 years ago

@psigen @mkoval Fixed collision check routine to use util.GetCollisionCheckPts. Fixed indentation & typo as well.

mkoval commented 8 years ago

@psigen Have you reviewed this yet?

mkoval commented 8 years ago

@psigen Could you please review this soon? I'd like to merge https://github.com/personalrobotics/prpy/pull/221, but it's blocked by this pull request.