mrpt-ros-pkg / mrpt_slam

ROS wrappers for SLAM algorithms in MRPT
http://wiki.ros.org/mrpt_slam
BSD 3-Clause "New" or "Revised" License
114 stars 49 forks source link

Noetic: mrpt_graphslam_2d fails to build saveAsEMFTwoMapsWithCorrespondences #73

Closed sloretz closed 5 months ago

sloretz commented 5 months ago

mrpt_graphslam_2d is failing to build on the ROS Noetic buildfarm:

08:10:26 /tmp/binarydeb/ros-noetic-mrpt-graphslam-2d-0.1.16/src/CMapMerger.cpp: In member function ‘void mrpt::graphslam::CMapMerger::mergeMaps()’:
08:10:34 /tmp/binarydeb/ros-noetic-mrpt-graphslam-2d-0.1.16/src/CMapMerger.cpp:530:26: error: ‘saveAsEMFTwoMapsWithCorrespondences’ is not a member of ‘mrpt::maps::COccupancyGridMap2D’
08:10:34   530 |     COccupancyGridMap2D::saveAsEMFTwoMapsWithCorrespondences(
08:10:34       |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
08:10:34 make[4]: Leaving directory '/tmp/binarydeb/ros-noetic-mrpt-graphslam-2d-0.1.16/.obj-x86_64-linux-gnu'

Likely caused by: https://github.com/MRPT/mrpt/commit/4c75ed7528644c597caa90a1967d002f63b0d572 released in https://github.com/ros/rosdistro/pull/40235

jlblancoc commented 5 months ago

Fixed via https://github.com/ros/rosdistro/pull/41494 @sloretz