forked from moveit/moveit_tutorials
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rebase to Master #2
Open
gautz
wants to merge
91
commits into
JeroenDM:ompl-constrained-planning-v2
Choose a base branch
from
gautz:ompl-constrained-planning-v2
base: ompl-constrained-planning-v2
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.
Open
Rebase to Master #2
gautz
wants to merge
91
commits into
JeroenDM:ompl-constrained-planning-v2
from
gautz:ompl-constrained-planning-v2
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
* Update the jog tutorials for new pkg/topic names * Update the rqt_graph image
Tutorial showing how to use GPD and Dex-Net within the MoveIt Task Constructor.
… to setup assistant tutorial (moveit#532)
Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
* Python3 migration of move_group_python_interface_tutorial * Python3 migration of collision_enviroments
- declare missing run dependencies - find_package(rviz_visual_tools) as we declare it as a build dependency - build lerp_motion_planner_plugin directly from all sources
- unused parameters, variables - uninitialized variables - out-of-order member initialization - unsigned int vs. signed int comparison
... to have latest geometric_shapes 0.7.2 available
* Whitepsace Cleanup * One more change
Update Branches to Noetic
Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
Co-authored-by: Bey Hao Yun <cardboardmails@gmail.com>
Co-authored-by: G.A. vd. Hoorn <g.a.vanderhoorn@tudelft.nl>
Co-authored-by: Takara Kasai <takara.kasai@gmail.com>
Co-authored-by: omid <omid@picknik.ai>
* Remove unnecessary comment on googling 'how to make python file executable', didn't get the expected result. For beginners, it can be time-saving
Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
…veit#437) * some changes related to those from trajopt in moveit_planners * clang format Co-authored-by: omid <omid@picknik.ai>
…t#501) * Create 'Representation and Evaluation of Constraints' tutorial * Integrate with new Approx Constraint Manifold tutorial * Update doc/planning_with_approximated_constraint_manifolds/planning_with_approximated_constraint_manifolds_tutorial.rst Co-authored-by: Mark Moll <Mark.Moll@gmail.com> * Update doc/planning_with_approximated_constraint_manifolds/planning_with_approximated_constraint_manifolds_tutorial.rst Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com> * Apply suggestions from code review Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com> Co-authored-by: Mark Moll <Mark.Moll@gmail.com> Co-authored-by: Felix von Drigalski <FvDrigalski@gmail.com>
* tutorial for debugging a failing test using docker * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com> * remove ros_distro env variable * fix grammar/spelling errors * Apply suggestions from code review * Fix sphynx bug Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com> Co-authored-by: Dave Coleman <dave@picknik.ai>
* Migrate to github actions * Fix formating errors
* fix visualization of the list Signed-off-by: mohamedsayed18 <mohamed95.a.s@gmail.com> Co-authored-by: Michael Görner <me@v4hn.de>
Here pre-commit run already. Is there any thing else to test before to merge? |
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
Checklist