ros-rolling-moveit-hybrid-planning (2.8.0-2jammy) jammy; urgency=high

  * Replaced numbers with SystemDefaultsQos() (#2271 <https://github.com/ros-planning/moveit2/issues/2271>)
  * Contributors: Shobuj Paul

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Sat, 09 Sep 2023 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.7.4-2jammy) jammy; urgency=high



 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 17 May 2023 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.7.3-2jammy) jammy; urgency=high

  * Replace check for the ROS_DISTRO env variable with a check for the rclcpp version (#2135 <https://github.com/ros-planning/moveit2/issues/2135>)
  * Replace Variable PROJECT_NAME in CMakeLists.txt with the actual name (#2020 <https://github.com/ros-planning/moveit2/issues/2020>)
  * Contributors: Jafar, Shobuj Paul

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Sun, 23 Apr 2023 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.7.2-2jammy) jammy; urgency=high

  * Switch from qos_event.hpp to event_handler.hpp (#2111 <https://github.com/ros-planning/moveit2/issues/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 <mailto: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 <mailto:jafar.uruc@gmail.com>
    Co-authored-by: Jafar <mailto:cafer.abdi@gmail.com>
    Co-authored-by: AndyZe <mailto:andyz@utexas.edu>
    Co-authored-by: Chris Thrasher <mailto:chrisjthrasher@gmail.com>
  * Update pre-commit (#2094 <https://github.com/ros-planning/moveit2/issues/2094>)
  * [hybrid planning] improve planning scene monitoring (#1090 <https://github.com/ros-planning/moveit2/issues/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 <mailto:sebastian.jahr@picknik.ai>
  * Contributors: AndyZe, Sebastian Jahr, Shobuj Paul

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Mon, 17 Apr 2023 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.7.1-2jammy) jammy; urgency=high

  * remove underscore from public member in MotionPlanResponse (#1939 <https://github.com/ros-planning/moveit2/issues/1939>)
    * remove underscore from private members
    * fix more uses of the suffix notation
  * Contributors: AlexWebb

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 22 Mar 2023 23:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.7.0-2jammy) jammy; urgency=high

  * Merge PR #1712 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/1844>)
  * Fix BSD license in package.xml (#1796 <https://github.com/ros-planning/moveit2/issues/1796>)
    * fix BSD license in package.xml
    * this must also be spdx compliant
  * Minimize use of this-> (#1784 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/1770>)
  * Add braces around blocks. (#999 <https://github.com/ros-planning/moveit2/issues/999>)
  * Use <> for non-local headers (#1734 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/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 <mailto:henningkayser@picknik.ai>
  * Contributors: Chris Thrasher, Christian Henkel, Cory Crean, Robert Haschke

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Sat, 28 Jan 2023 23:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.6.0-2jammy) jammy; urgency=high

  * Parallel planning pipelines (#1420 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/1550>)
  * Contributors: Robert Haschke, Sebastian Jahr

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 09 Nov 2022 23:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.5.3-2jammy) jammy; urgency=high



 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 27 Jul 2022 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.5.2-2jammy) jammy; urgency=high

  * Merge remote-tracking branch 'origin/main' into feature/msa
  * Launch file cleanup (#1380 <https://github.com/ros-planning/moveit2/issues/1380>)
  * Remove unnecessary rclcpp.hpp includes (#1333 <https://github.com/ros-planning/moveit2/issues/1333>)
  * [Hybrid Planning] Improve action cancellation (#1272 <https://github.com/ros-planning/moveit2/issues/1272>)
  * Contributors: AndyZe, Jafar, Vatan Aksoy Tezer

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Sun, 17 Jul 2022 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.5.1-2jammy) jammy; urgency=high

  * Remove position controllers from CMake (#1285 <https://github.com/ros-planning/moveit2/issues/1285>)
  * Contributors: Vatan Aksoy Tezer

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Mon, 30 May 2022 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.5.0-2jammy) jammy; urgency=high

  * Fix hybrid planning launching (#1271 <https://github.com/ros-planning/moveit2/issues/1271>)
  * Enable cppcheck (#1224 <https://github.com/ros-planning/moveit2/issues/1224>)
    Co-authored-by: jeoseo <mailto:jeongwooseo2012@gmail.com>
  * Make moveit_common a 'depend' rather than 'build_depend' (#1226 <https://github.com/ros-planning/moveit2/issues/1226>)
  * Avoid bind(), use lambdas instead (#1204 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/1181>)
  * Enable rolling / jammy CI (again) (#1134 <https://github.com/ros-planning/moveit2/issues/1134>)
    * Use ros2_control binaries
    * Use output screen instead of explicitly stating stderr
  * [hybrid planning] Adjust planning scene locking (#1087 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/1052>)
  * [hybrid planning] Use a map of expected feedback codes (#1065 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <mailto:tylerjw@gmail.com>
  * [hybrid planning] Add action abortion and test; improve the existing test (#980 <https://github.com/ros-planning/moveit2/issues/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 <mailto:tyler@picknik.ai>
  * [hybrid planning] Delete the pass-through option (#986 <https://github.com/ros-planning/moveit2/issues/986>)
    * Delete the pass-through option
    * Suppress clang warning
    * Handle (waypoint_count < 0) possibility
    Co-authored-by: Tyler Weaver <mailto:tyler@picknik.ai>
  * Remove unused parameters. (#1018 <https://github.com/ros-planning/moveit2/issues/1018>)
    Co-authored-by: Tyler Weaver <mailto:tyler@picknik.ai>
    Co-authored-by: Vatan Aksoy Tezer <mailto:vatan@picknik.ai>
  * Contributors: AndyZe, Cory Crean, David V. Lu!!, Henning Kayser, Jafar, Vatan Aksoy Tezer, jeoseo, v4hn

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 25 May 2022 22:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.4.0-2jammy) jammy; urgency=high

  * hybrid_planning: Fix global_planner action name (#960 <https://github.com/ros-planning/moveit2/issues/960>)
  * Put hybrid planning actions under a common namespace (#932 <https://github.com/ros-planning/moveit2/issues/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 <mailto: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 <mailto:cafer.abdi@gmail.com>
  * Contributors: AndyZe

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 19 Jan 2022 23:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.3.2-2jammy) jammy; urgency=high

  * Fix syntax (#939 <https://github.com/ros-planning/moveit2/issues/939>)
    Co-authored-by: AndyZe <mailto:zelenak@picknik.ai>
  * Contributors: Sebastian Jahr

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Tue, 28 Dec 2021 23:00:00 -0000

ros-rolling-moveit-hybrid-planning (2.3.1-2jammy) jammy; urgency=high

  * Bump new packages to 2.3.0
  * Add codespell to precommit, fix A LOT of spelling mistakes (#934 <https://github.com/ros-planning/moveit2/issues/934>)
  * CMake fix and cleanup abstract class (#6 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/585>)
    Fix hybrid planning include folders (#675 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/426>)
    * Add forward_trajectory local solver plugin (#359 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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 <https://github.com/ros-planning/moveit2/issues/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

 -- MoveIt Release Team <moveit_releasers@googlegroups.com>  Wed, 22 Dec 2021 23:00:00 -0000


