Changelog for package moveit_ros_perception

2.9.0 (2024-01-09)

• (moveit_ros) add missing CYLINDER check (#2640) As this check was missing, the CYLINDER check later on was unreachable
• Node logging in moveit_core (#2503)
• 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
• Update clang-format-14 with QualifierAlignment (#2362)
  • Set qualifier order in .clang-format
  • Ran pre-commit to update according to new style guide

• 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, Matthijs van der Burgh, Sebastian Jahr, Shobuj Paul, Tyler Weaver

2.8.0 (2023-09-10)

• Replaced numbers with SystemDefaultsQos() (#2271)
• Use [isnan]{.title-ref} in [DepthImageOctomapUpdater]{.title-ref} (#2201)
• Contributors: Ezra Brooks, Shobuj Paul

2.7.4 (2023-05-18)

2.7.3 (2023-04-24)

• Replace check for the ROS_DISTRO env variable with a check for the rclcpp version (#2135)
• Replace Variable PROJECT_NAME in CMakeLists.txt with the actual name (#2020)
• Contributors: Jafar, Shobuj Paul

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)

• Contributors: Sebastian Jahr, Shobuj Paul

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>

• Contributors: Robert Haschke

2.7.0 (2023-01-29)

• converted characters from string format to character format (#1881)
• Fix BSD license in package.xml (#1796)
  • fix BSD license in package.xml
  • this must also be spdx compliant
• Enable [-Wold-style-cast]{.title-ref} (#1770)
• Use sensor data QOS profile for sensor_msgs::Image and sensor_msgs::PointCloud (#664) Co-authored-by: Henry Moore <henrygerardmoore@gmail.com> Co-authored-by: Tyler Weaver <tyler@picknik.ai>
• Remove [MOVEIT_LIB_NAME]{.title-ref} (#1751) It\'s more readable and searchable if we just spell out the target name.
• 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
• 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, Chris Thrasher, Christian Henkel, Cory Crean, Nathan Brooks, Robert Haschke, Sameer Gupta

2.6.0 (2022-11-10)

• Merge PR #1553: Improve cmake files
• Use standard exported targets: export_\${PROJECT_NAME} -> \${PROJECT_NAME}Targets
• Improve CMake usage (#1550)
• Remove __has_include statements (#1481)
• Contributors: Robert Haschke, Sebastian Jahr, Vatan Aksoy Tezer

2.5.3 (2022-07-28)

• pointcloud_octomap_updater: Don\'t return false when not finding optional parameter (#1418)

  • Don\'t return false when not finding optional parameter

  * Update moveit_ros/perception/pointcloud_octomap_updater/src/pointcloud_octomap_updater.cpp Co-authored-by: AndyZe <andyz@utexas.edu> Co-authored-by: AndyZe <andyz@utexas.edu> Co-authored-by: Vatan Aksoy Tezer <vatan@picknik.ai>

• Contributors: Jafar

2.5.2 (2022-07-18)

• Remove no longer needed no-pedantic option in moveit_ros_occupancy_map_monitor package (#1397)
• Merge remote-tracking branch \'origin/main\' into feature/msa
• Removing more boost usage (#1372)
• Merge remote-tracking branch \'upstream/main\' into feature/msa
• Removing some boost usage (#1331) Co-authored-by: Vatan Aksoy Tezer <vatan@picknik.ai>
• Remove unnecessary rclcpp.hpp includes (#1333)
• Update plugin library paths (#1304)
• Merge pull request #3106 from v4hn/pr-master-bind-them-all / banish bind()
• Fix clang-tidy
• banish bind()
• various: prefer objects and references over pointers
• Contributors: David V. Lu, Henry Moore, Jafar, Michael G