forked from moveit/moveit
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Ompl constrained orientation #9
Draft
gautz
wants to merge
223
commits into
JeroenDM:ompl-orientation-constraints-v5
Choose a base branch
from
gautz:ompl-constrained-orientation
base: ompl-orientation-constraints-v5
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Ompl constrained orientation #9
gautz
wants to merge
223
commits into
JeroenDM:ompl-orientation-constraints-v5
from
gautz:ompl-constrained-orientation
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n the python interface (moveit#2231)
* Move last_sent_command_ update from constructor to start * Tweak demo configs * Add install folders to CMakeLists * Add clarity to parameter warning
* Add a utility for logging collision pairs * Update printing style, rename function, move constants into the struct
... until it is released.
This reverts commit 4db7ccb. Instead, use ros-testing repo.
…it#2293) Otherwise, we get entries like this in ompl_planning.yaml, where "None", of course, is not a valid planner config: arm: default_planner_config: None
…oveit#2300) This fixes a failing test on the ROS buildfarm: http://build.ros.org/job/Ndev__moveit__ubuntu_focal_amd64/1
* Rework velocity limit enforcement, add unit test * Test every command out of Servo * Minor variable renaming * Clang format & clang tidy * Delete unused acceleration variable
- better use of space / overall smaller panel - remove goal constraints field, it's not used and not intuitive in the GUI either - reorder check boxes by subjective usefulness - Add better tooltips - Remove autoScroll from Planning Parameters field - Fix tabstop order - Align columns of Planning tab Co-authored-by: Robert Haschke <[email protected]>
... to correctly rewrite shebang
fix some more python3 issues
moveit/geometric_shapes#161 introduced a syntax simplification employing C++14 features (constexpr).
* fixed mesh_filter error spam on startup * infinite loop fix from Andy
Deprecate namespace moveit::planning_interface:: in favor of moveit_cpp::
* Format default pipeline / planner as bold Co-Authored-By: Robert Haschke <[email protected]>
So far, executeAndMonitor() was only called via planAndExecuteHelper(), which was resetting the preempt-requested flag when entering the function (ignoring leftover preempt signals from previous runs). However, now MTC also calls executeAndMonitor() directly and thus we need to reset the flag there as well - but only if it is called directly.
* Publish Servo stop in Pose Tracking cleanup * Move publish halt to stopMotion()
* Order PlaceLocations by quality during planning The quality field was added by moveit/moveit_msgs@cdb9e4a. The changes are adapted from pick.cpp, where the Grasps are also ordered by quality. * Use stable_sort to preserve order of equal-quality place locations * Pick: use stable_sort to preserve order of equal-quality grasps Co-authored-by: Robert Haschke <[email protected]>
Occurred when a group was defined in the launch file without a `ns` attribute.
Provide python bindings for basic classes of moveit_core: - collision_detection: Contact, CollisionRequest, CollisionResult, AllowedCollisionMatrix - kinematic_constraints: ConstraintEvaluationResult, KinematicConstraintSet, constructGoalConstraints - planning_scene: PlanningScene, - robot_model: RobotModel, JointModelGroup - robot_state: RobotState - transforms: Transforms
This is public API because the solver's getPositionIK requires input poses in the frame and the relevant transform is otherwise inaccessible. Partially reverts moveit#2461.
gautz
changed the base branch from
orientation-constraints-parameterization
to
ompl-orientation-constraints-v5
April 9, 2021 08:13
2 tasks
It seems to work! I updated the corresponding tutorial. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I got orientation constraints back available with OMPL constrained planner. Here is what I did:
Related issues:
Checklist