CHANGELOG
Changelog for package moveit_hybrid_planning
2.11.0 (2024-09-16)
2.10.0 (2024-06-13)
Migrate ros-planning org to moveit (#2847) * Rename github.com/ros-planning -> github.com/moveit * Rename ros-planning.github.io -> moveit.github.io * Rename ros-planning organization in docker and CI workflow files - ghcr.io/ros-planning -> ghcr.io/moveit - github.repository == ‘moveit/*’’
Remove HP error codes interface (#2774)
Unify log names (#2720) Co-authored-by: Abishalini Sivaraman <abi.gpuram@gmail.com>
CMake format and lint in pre-commit (#2683)
Clean-up hybrid planning package (#2603)
Contributors: Robert Haschke, Sebastian Jahr, Tyler Weaver
2.9.0 (2024-01-09)
Update ros2_control usage (#2620) * Update ros2_control usage * Update xacro file
Node logging for the rest of MoveIt (#2599)
[Planning Pipeline Refactoring] #2 Enable chaining planners (#2457) * Enable chaining multiple planners
[Planning Pipeline Refactoring] #1 Simplify Adapter - Planner chain (#2429)
Use node logging in moveit_ros (#2482)
Add new clang-tidy style rules (#2177)
Update pre-commit and add to .codespell_words (#2465)
Merge branch ‘main’ into dependabot/github_actions/SonarSource/sonarcloud-github-c-cpp-2
Remove old deprecated functions (#2384)
Converts float to double (#2343) * Limiting the scope of variables #874 Limited the scope of variables in moveit_core/collision_detection * Update moveit_core/collision_detection/src/collision_octomap_filter.cpp Co-authored-by: AndyZe <andyz@utexas.edu> * Update moveit_core/collision_detection/src/collision_octomap_filter.cpp Co-authored-by: AndyZe <andyz@utexas.edu> * Update moveit_core/collision_detection/src/collision_octomap_filter.cpp Co-authored-by: AndyZe <andyz@utexas.edu> * convert float to double * change double to float * Feedback fixes * Introduced variables removed from previous merge commit * Updated GL_Renderer function definitions with double instead of float * Changed update() function arguments to float since it is a derived virtual function and needs to be overriden * Fixed all override errors in visualization * Fixed override errors in perception *Changed reinterpret_cast to double from float* * change variable types to fit function definition * Fixed clang-tidy warnings * Fixed scope of reusable variables ——— Co-authored-by: Salah Soliman <salahsoliman96@gmail.com> Co-authored-by: AndyZe <andyz@utexas.edu> Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
Merge branch ‘main’ into dependabot/github_actions/SonarSource/sonarcloud-github-c-cpp-2
Contributors: Marq Rasmussen, Rayene Messaoud, Sebastian Jahr, Shobuj Paul, Tyler Weaver
2.8.0 (2023-09-10)
Replaced numbers with SystemDefaultsQos() (#2271)
Contributors: Shobuj Paul
2.7.4 (2023-05-18)
2.7.3 (2023-04-24)
2.7.2 (2023-04-18)
Switch from qos_event.hpp to event_handler.hpp (#2111) * Switch from qos_event.hpp to event_handler.hpp * moveit_common: Add a cmake interface library to keep humble support on main * Include qos_event.hpp or event_handler.hpp depending on the ROS 2 version * Fix ament_lint_cmake * Fix clang-tidy * PRIVATE linking in some cases * Update moveit_common/cmake/moveit_package.cmake Co-authored-by: Chris Thrasher <chrisjthrasher@gmail.com> * Fix servo and cleanup excessive CMake variable usage * Cleanup & make compiling * Small variable naming and const cleanup * Restore OpenCV linking * Public/private linking fixup * Revert “Restore OpenCV linking” This reverts commit 57a9efa806e59223e35a1f7e998d7b52f930c263. ——— Co-authored-by: JafarAbdi <jafar.uruc@gmail.com> Co-authored-by: Jafar <cafer.abdi@gmail.com> Co-authored-by: AndyZe <andyz@utexas.edu> Co-authored-by: Chris Thrasher <chrisjthrasher@gmail.com>
Update pre-commit (#2094)
[hybrid planning] improve planning scene monitoring (#1090) * Create new PSM in local constraint solver. Different type of collision checking. * Small boolean logic fixup * Don’t configure planning scene monitor twice and pass scene as const * Remove not required call of updateSceneWithCurrentState() * Revert PlanningSceneMonitorConstPtr to PlanningSceneMonitorPtr * Set planning_scene_monitor update rate * Decrease planning scene update rate * Use updateSceneWithCurrentState() in psm * Revert the manner of collision checking ——— Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>
Contributors: AndyZe, Sebastian Jahr, Shobuj Paul
2.7.1 (2023-03-23)
remove underscore from public member in MotionPlanResponse (#1939) * remove underscore from private members * fix more uses of the suffix notation
Contributors: AlexWebb
2.7.0 (2023-01-29)
Merge PR #1712: fix clang compiler warnings + stricter CI
Add default constructors … as they are not implicitly declared anymore
Explicitly declare overrides
Add default copy/move constructors/assignment operators As a user-declared destructor deletes any implicitly-defined move constructor/assignment operator, we need to declared them manually. This in turn requires to declare the copy constructor/assignment as well.
Fix more clang warnings
Fix -Wdelete-non-abstract-non-virtual-dtor
Cleanup msg includes: Use C++ instead of C header (#1844)
Fix BSD license in package.xml (#1796) * fix BSD license in package.xml * this must also be spdx compliant
Minimize use of this-> (#1784) It’s often unnecessary. MoveIt already avoids this in most cases so this PR better cements that existing pattern.
Enable -Wold-style-cast (#1770)
Add braces around blocks. (#999)
Use <> for non-local headers (#1734) Unless a header lives in the same or a child directory of the file including it, it’s recommended to use <> for the #include statement. For more information, see the C++ Core Guidelines item SF.12 https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#sf12-prefer-the-quoted-form-of-include-for-files-relative-to-the-including-file-and-the-angle-bracket-form-everywhere-else
Fix clang-tidy issues (#1706) * Blindly apply automatic clang-tidy fixes * Exemplarily cleanup a few automatic clang-tidy fixes * Clang-tidy fixups * Missed const-ref fixups * Fix unsupported non-const -> const * More fixes Co-authored-by: Henning Kayser <henningkayser@picknik.ai>
Contributors: Chris Thrasher, Christian Henkel, Cory Crean, Robert Haschke
2.6.0 (2022-11-10)
Parallel planning pipelines (#1420) * Add setTrajectoryConstraints() to PlanningComponent * Add planning time to PlanningComponent::PlanSolution * Replace PlanSolution with MotionPlanResponse * Address review * Add MultiPipelinePlanRequestParameters Add plan(const MultiPipelinePlanRequestParameters& parameters) Add mutex to avoid segfaults Add optional stop_criterion_callback and solution_selection_callback Remove stop_criterion_callback Make default solution_selection_callback = nullptr Remove parameter handling copy&paste code in favor of a template Add TODO to refactor pushBack() method into insert() Fix selection criteria and add RCLCPP_INFO output Changes due to rebase and formatting Fix race condition and segfault when no solution is found Satisfy clang tidy Remove mutex and thread safety TODOs Add stopping functionality to parallel planning Remove unnecessary TODOs * Fix unused plan solution with failure * Add sanity check for number of parallel planning problems * Check stopping criterion when new solution is generated + make thread safe * Add terminatePlanningPipeline() to MoveItCpp interface * Format! * Bug fixes * Move getShortestSolution callback into own function * No east const * Remove PlanSolutions and make planner_id accessible * Make solution executable * Rename update_last_solution to store_solution * Alphabetize includes and include plan_solutions.hpp instead of .h * Address review * Add missing header * Apply suggestions from code review Co-authored-by: AndyZe <andyz@utexas.edu> Co-authored-by: AndyZe <andyz@utexas.edu>
Merge PR #1553: Improve cmake files
Cleanup cmake files - Replace ament_export_libraries() -> ament_export_targets(HAS_LIBRARY_TARGET) - Replace ament_export_include_directories() -> INCLUDES DESTINATION include See https://docs.ros.org/en/foxy/How-To-Guides/Ament-CMake-Documentation.html#building-a-library
Use standard exported targets: export_${PROJECT_NAME} -> ${PROJECT_NAME}Targets
Improve CMake usage (#1550)
Contributors: Robert Haschke, Sebastian Jahr
2.5.3 (2022-07-28)
2.5.2 (2022-07-18)
2.5.1 (2022-05-31)
Remove position controllers from CMake (#1285)
Contributors: Vatan Aksoy Tezer
2.5.0 (2022-05-26)
Fix hybrid planning launching (#1271)
Enable cppcheck (#1224) Co-authored-by: jeoseo <jeongwooseo2012@gmail.com>
Make moveit_common a ‘depend’ rather than ‘build_depend’ (#1226)
Avoid bind(), use lambdas instead (#1204) Adaption of https://github.com/ros-planning/moveit/pull/3106
banish bind() source:https://github.com/ros-planning/moveit/pull/3106/commits/a2911c80c28958c1fce8fb52333d770248c4ec05; required minor updates compared to original source commit in order to ensure compatibility with ROS2
RCLCPP Upgrade Bugfixes (#1181)
Enable rolling / jammy CI (again) (#1134) * Use ros2_control binaries * Use output screen instead of explicitly stating stderr
[hybrid planning] Adjust planning scene locking (#1087) * Create a copy of the planning scene. const robot state. * Use LockedPlanningSceneRO over lockSceneRead() * Use lambda function
[Hybrid Planning] configurable planning scene topics (#1052)
[hybrid planning] Use a map of expected feedback codes (#1065) * Use a map of expected feedback codes * Use a constexpr function instead of unordered_map * Don’t need this #include * Minor function renaming
Disable hybrid planning test, don’t cache ci docker at all and don’t cache upstream_ws if repos file is changed (#1051) * Don’t cache docker builds * Don’t cache upstream ws * Use new action for getting the latest timestamp .repos file has been edited * Debug * Fix repos path * Disable hybrid planning test * Use more verbose name Co-authored-by: Tyler Weaver <tylerjw@gmail.com>
[hybrid planning] Add action abortion and test; improve the existing test (#980) * Add action abortion and test; improve the existing test * Add controller run-dependency * Fix the clearing of robot trajectory when a collision would occur * Fix replanning if local planner is stuck * Lambda function everything * Thread safety for stop_hybrid_planning_ * Thread-safe state_ * Clang tidy * Update the planning scene properly * Update Servo test initial_positions.yaml Co-authored-by: Tyler Weaver <tyler@picknik.ai>
[hybrid planning] Delete the pass-through option (#986) * Delete the pass-through option * Suppress clang warning * Handle (waypoint_count < 0) possibility Co-authored-by: Tyler Weaver <tyler@picknik.ai>
Remove unused parameters. (#1018) Co-authored-by: Tyler Weaver <tyler@picknik.ai> Co-authored-by: Vatan Aksoy Tezer <vatan@picknik.ai>
Contributors: AndyZe, Cory Crean, David V. Lu!!, Henning Kayser, Jafar, Vatan Aksoy Tezer, jeoseo, v4hn
2.4.0 (2022-01-20)
hybrid_planning: Fix global_planner action name (#960)
Put hybrid planning actions under a common namespace (#932) * Put hybrid planning actions under a common namespace * Use ~ * New pkg for common resources. Does not work. * Use inline rather than anonymous namespace Co-authored-by: Jafar Abdi <cafer.abdi@gmail.com> * Revert “Use inline rather than anonymous namespace” This reverts commit 29a7d279776be21f4666c7e0fadeaa6b7ef8debf. * Revert “New pkg for common resources. Does not work.” This reverts commit 68a173baee4b7f8b2c1f74285f96c8e3892c5909. * Some progress toward loading common parameters Co-authored-by: Jafar Abdi <cafer.abdi@gmail.com>
Contributors: AndyZe
2.3.2 (2021-12-29)
Fix syntax (#939) Co-authored-by: AndyZe <zelenak@picknik.ai>
Contributors: Sebastian Jahr
2.3.1 (2021-12-23)
Bump new packages to 2.3.0
Add codespell to precommit, fix A LOT of spelling mistakes (#934)
CMake fix and cleanup abstract class (#6) * Install hybrid_planning_demo_node separately to avoid exporting it * Fix exported include directory * Remove getTargetWayPointIndex() from abstract trajectory operator class * Delete unused getTargetWayPointIndex()
Standardize with moveit_cpp parameters. Fix some parameter loading errors
Enable reaction to planner failure in the planner logic (#3) * Add unsuccessful action Hybrid Planning events and handle them in logic * Replace std::once with simple bool variable * Remove unneeded variable and update comments Don’t const& for built-in types Co-authored-by: Tyler Weaver <tylerjw@gmail.com>
Update robot state properly Update robot state properly; remove debug prints; clang format Check if the local planner is stuck within the forward_traj plugin, not local_planner_component
Pass GlobalPlanner failing to HybridPlanningManager
Move common launch elements to a Python file, for easy re-use Refactor Global and Local Planner Components into NodeClasses Add a simple launch test (#1) Try to fix plugin export; add helpful debug when stuck Error if global planning fails READY and AWAIT_TRAJ states are redundant Lock the planning scene as briefly as possible Specify joint group when checking waypoint distance Implement a reset() function in the local planner Detect when the local planner gets stuck
Add generic global planner plugin, support MotionSequenceRequest (#585) Fix hybrid planning include folders (#675) Order stuff in the CMakeLists.txt and remove control_box package Update README Move member initialization to initializer lists Remove control_box include dependency Replace “loop-run” with “iteration” Remove cpp interface class constructors and destructors Use joint_state_broadcaster, clean up test node, add execution dependencies Use only plugin interface header files and add missing dependencies Clean up constructor and destructor definitions Declare missing parameter in moveit_planning_pipeline_plugin Move rclcpp::Loggers into anonymous namespaces Switch CI branches to feature/hybrid_planning Update message name Remove moveit_msgs from .repos file Update github workflows Remove note from readme about building from source Minor renamings, make reset() noexcept Check for vector length of zero Load moveit_cpp options with the Options helper. Reduces LOC. Set the planning parameters within plan() Function renaming Authors and descriptions in header files only. New header file for error code interface. Update namespacing Use default QoS for subscribers Better progress comparison Add publish_joint_positions and publish_joint_velocities param Grammar and other minor nitpicks Restore moveit_msgs to .repos, for now
Refactor local planner plugins (#447) * Add reset method for trajectory operator and local constraint sampler * Refactor next_waypoint_sampler into simple_sampler * Include collision checking to forward_trajectory and remove unneeded plugin * Fix formatting and plugin description * Update README and add missing planner logic plugin description Add TODO to use lifecycle components nodes to trigger initialization Return a reaction result instead of bool on react() Set invalidation flag to false on reset() in ForwardTrajectory local solver Return local planner feedback from trajectory operator function calls Fix segfault caused by passing through the global trajectory Update comment, unify logging and add missing config parameters Update to rolling
Restructure hybrid_planning package (#426) * Add forward_trajectory local solver plugin (#359) * Use ros2_control based joint simulation and remove unnecessary comment * Update copyrights * Restructure hybrid planning package * Fix formatting and add missing time stamp in local solver plugin * Remove unnecessary logging and param loading * Enable different interfaces between local planner and controller * Use JointGroupPositionController as demo hardware controller
Code cleanup & MoveIt best practices (#351) * Export missing plugins * Use std::chrono_literals * Construct smart pointers with std::make_* instead of ‘new’ * Fixup const-ref in function signatures, reduce copies * Improve planning scene locking, robot state processing, controller logic
Refine local planner component (#326) * Make local planner component generic * Add next_waypoint_sampler trajectory operator * Update hybrid planning test to include collision object * Clean up code and fix minor bugs. * Update local planner component parameter * Add local collision constraint solver * Update planning scene topic name and test node * Fix bugs and runtime errors in local planner component and it’s plugins * Add collision object that invalidates global trajectory * Add simple “stop in front of collision object” demo * Add hybrid planning manager reaction to local planner feedback * Fix ament_lint_cmake * Ensure that local planner start command and collision event are send once * Add simple replanning logic plugin * Use current state as start state for global planner * Use RobotTrajectory instead of constraint vector describe local problem * Add PlanningSceneMonitorPtr to local solver plugin * Add local planner frequency parameter * Use PID controller to create control outputs for robot controller * Add hybrid_planning_manager config file * Add more complex test node * Update README * Reset index in next_waypoint_sampler * Use correct isPathValid() interface * Rename path_invalidation flag * Read planning scene instead of cloning it in local planner * Add TODO creator * Rename local constraint solver plugin * Use read-locked access to the planning scene for collision checking * Rename constraint_solver into local_constraint_solver * Add missing pointer initialization
Hybrid planning architecture (#311) * Add hybrid_planning architecture skeleton code * Add simple state machines to hybrid planning manager and local planner * Initial global planner prototype implementation * Forward joint_trajectory with local planner * Forward hybrid planning motion request to global planner * Abstract planner logic from hybrid planning manager by using a plugin * Implement single plan execution logic * Add test launch files, RViz and demo config * Add test for motion_planning_request
Contributors: AndyZe, David V. Lu!!, Henning Kayser, Jens Vanhooydonck, Sebastian Jahr