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 <[email protected]>
* 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 <[email protected]>
Co-authored-by: Felix von Drigalski <[email protected]>
Co-authored-by: Bey Hao Yun <[email protected]>
Co-authored-by: G.A. vd. Hoorn <[email protected]>
Co-authored-by: Takara Kasai <[email protected]>
Co-authored-by: omid <[email protected]>
* 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 <[email protected]>
…veit#437) * some changes related to those from trajopt in moveit_planners * clang format Co-authored-by: omid <[email protected]>
…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 <[email protected]> * Update doc/planning_with_approximated_constraint_manifolds/planning_with_approximated_constraint_manifolds_tutorial.rst Co-authored-by: Felix von Drigalski <[email protected]> * Apply suggestions from code review Co-authored-by: Felix von Drigalski <[email protected]> Co-authored-by: Mark Moll <[email protected]> Co-authored-by: Felix von Drigalski <[email protected]>
* tutorial for debugging a failing test using docker * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <[email protected]> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <[email protected]> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <[email protected]> * Update doc/test_debugging/test_debugging_tutorial.rst Co-authored-by: Robert Haschke <[email protected]> * remove ros_distro env variable * fix grammar/spelling errors * Apply suggestions from code review * Fix sphynx bug Co-authored-by: Robert Haschke <[email protected]> Co-authored-by: Dave Coleman <[email protected]>
* Migrate to github actions * Fix formating errors
* fix visualization of the list Signed-off-by: mohamedsayed18 <[email protected]> Co-authored-by: Michael Görner <[email protected]>
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