Changelog for package moveit_ros_move_group

2.8.0 (2023-09-10)

• Replaced boost::algorithm::join with fmt::join (#2273)

  • Replaced boost::algorithm::join with fmt::join
  • Made changes in CMakeLists.txt to accomodate fmt
  • Updated package.xml files
  • removed redundant boost dependencies

  * Rename variables -> variable ---------Co-authored-by: Sebastian Castro <4603398+sea-bass@users.noreply.github.com> Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>

• Specify controller name in MGI execution (#2257)

  • Specify controller name in MGI execute

  * Finish comment ---------Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>

• fix move_group capabilities loading (#2270)

  • fix move_group capabilities loading
  • clang-format

• Cleanup move_group CMake (#2226)

• Contributors: Shobuj Paul, Stephanie Eng, Tyler Weaver, Yang Lin

2.7.4 (2023-05-18)

• Fix MoveGroup action cancel callback (#2118) Moves the execution callback into its own thread to avoid blocking and actually calls the preempt function in with the cancel callback.
• Scale acceleration and velocity of cartesian interpolations (#1968)
• Contributors: Jonathan Grebe, Yadu

2.7.3 (2023-04-24)

• Replace Variable PROJECT_NAME in CMakeLists.txt with the actual name (#2020)
• Contributors: Shobuj Paul

2.7.2 (2023-04-18)

• Fix MoveItCpp issues (port from MoveIt1) (#2001) * Fix MoveitCpp\'s const member accessors They should return a ConstPtr instead of a const Ptr&!

  • Fix SEVERE ClassLoader warning when releasing MoveItCpp
  • PSM was released before copy of its RobotModel -> removed extra RobotModel copy
  • clearContents() was broken:
  • resets in wrong order: psm_ should be last

  - trajectory_execution_manager_ was missing I suggest to omit clearContents() and rely on the (correct) ordering of member variables. While this is not explicit, we ensure that we don\'t miss any newly added member variable. Fix: https://github.com/ros-planning/moveit2/issues/1597 ---------Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai> Co-authored-by: Jafar <cafer.abdi@gmail.com> Co-authored-by: Sebastian Jahr <sebastian.jahr@tuta.io> Co-authored-by: JafarAbdi <jafar.uruc@gmail.com>

• Contributors: Robert Haschke

2.7.1 (2023-03-23)

• Fix member naming (#1949) * Update clang-tidy rules for readability-identifier-naming Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>

• remove underscore from public member in MotionPlanResponse (#1939)

  • remove underscore from private members
  • fix more uses of the suffix notation

• Contributors: AlexWebb, Robert Haschke

2.7.0 (2023-01-29)

• move_group: Delete unused execute_trajectory_service_capability (#1836)
• keep printf color change on same line (#1828) This ensures the color reset is applied because printing the color reset after new lines seems to preven the color from actually being reset. Co-authored-by: William Wedler <william.wedler@resquared.com>
• Fix BSD license in package.xml (#1796)
  • fix BSD license in package.xml
  • this must also be spdx compliant
• Add braces around blocks. (#999)
• Used C++ style cast instead of C style cast (#1628) Co-authored-by: Henning Kayser <henningkayser@picknik.ai>

• 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: Abhijeet Das Gupta, AndyZe, Christian Henkel, Cory Crean, Robert Haschke, Will

2.6.0 (2022-11-10)

• Short-circuit planning adapters (#1694) * Revert \"Planning request adapters: short-circuit if failure, return code rather than bool (#1605)\" This reverts commit 66a64b4a72b6ddef1af2329f20ed8162554d5bcb.

  • Add debug message in call stack of planning_request_adapters
  • Short-circuit planning request adapters
  • Replace if-elseif cascade with switch
  • Cleanup translation of MoveItErrorCode to string
  • Move default code to moveit_core/utils
  • Override defaults in existing getActionResultString()
  • Provide translations for all error codes defined in moveit_msgs
  • Fix comment according to review

  * Add braces Co-authored-by: Henning Kayser <henningkayser@picknik.ai> * Add braces Co-authored-by: Henning Kayser <henningkayser@picknik.ai> Co-authored-by: Henning Kayser <henningkayser@picknik.ai> Co-authored-by: AndyZe <andyz@utexas.edu>

• avoid implicit conversions (#1593)

• Remove unused header (#1572)

• Merge PR #1553: Improve cmake files

• Use standard exported targets: export_\${PROJECT_NAME} -> \${PROJECT_NAME}Targets

• Improve CMake usage (#1550)

• Removed plan_with_sensing (#1142)

• Remove __has_include statements (#1481)

• Contributors: Abishalini Sivaraman, Robert Haschke, Sarah Nix, Sebastian Jahr, Stephanie Eng, Vatan Aksoy Tezer

2.5.3 (2022-07-28)

2.5.2 (2022-07-18)

• Merge pull request #1402 from Abishalini/pr-sync-a436a97 Sync with MoveIt
• Merge https://github.com/ros-planning/moveit/commit/a436a9771f7445c162cc3090c4c7c57bdb5bf194
• Merge remote-tracking branch \'origin/main\' into feature/msa
• Remove manipulation from moveit_ros (#1177)
• Merge pull request #3137 from TAMS-Group/pr-master-monitor-dynamics move_group can optionally monitor dynamics
• optionally enable dynamics monitoring in move_group node
• convert move_group to LOGNAME
• Merge pull request #3106 from v4hn/pr-master-bind-them-all / banish bind()
• banish bind()
• Contributors: Abishalini, Michael G