-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ConfigurationToTSRwithTrajectoryConstraint #566
base: gilwoo/planner_dart
Are you sure you want to change the base?
Add ConfigurationToTSRwithTrajectoryConstraint #566
Conversation
…RwithTrajectoryConstraintPlanner
I haven't thought about this very much, but I initially thought it might make sense to add trajectory constraints as a part of all Also, I think it would be good to separate the new |
…ido into gilwoo/ConfigurationToTSRwTrajConstraint
…:personalrobotics/aikido into gilwoo/ConfigurationToTSRwTrajConstraint
Thanks for the comments @brianhou !
Yep I agree on both of these. I mainly chose this way for the latter challenge you mentioned and also because this is the only constrained planning problem type we use now. I think it's worth revisiting for a more general solution once we have more of these constrained planning problem types. |
In preparation for #467, this PR contains the following update:
planner::dart::ConfigurationToTSRwithTrajectoryConstraint
planner::dart::CRRTConfigurationToTSRwithTrajectoryConstraintPlanner
planner_dart
component and make relevant changes in CMakeLists files.The last change is necessary because
CRRTConfigurationToTSRwithTrajectoryConstraintPlanner
needs to callplanner_ompl
, which needs to be built before this planner is compiled.Testing
This PR passes existing unit tests. I'll add unit tests for this planner after the initial review.
Before creating a pull request
make format
Before merging a pull request
CHANGELOG.md