Skip to content

sick_scan_Base: using fork

Fabian Reister requested to merge feature/fork-sick_scan_base into main

There were multiple issues related to the sick_scan_base integration:

  • in the original sick_scan_base implementation, custom C++ preprocessor definitions of PI and deg2rad were defined and caused conflicts with Simox. I removed them and used the existing constants M_PI instead
  • The sick_scan_base cmake project didn't define an installation step. Finding the package was done manually (see https://gitlab.com/ArmarX/RobotAPI/-/merge_requests/280 Findsick_scan_base.cmake) => I added the installation step and removed the custom search stuff.

Merge request reports