Unable to compile rtabmap for ROS

[matlabbe]

From @GoogleCodeExporter on May 26, 2015 18:56

What steps will reproduce the problem?

1. compiling rtabmap for ROS with current revisions of ros-pkg and rtabmap

What is the expected output? What do you see instead?

[ 58%] Building CXX object 
rtabmap/CMakeFiles/rtabmap_ros.dir/src/rviz/MapCloudDisplay.cpp.o
/home/dominik/catkin_hydro/src/rtabmap/src/rviz/MapCloudDisplay.cpp: In member 
function ‘void rtabmap::MapCloudDisplay::processMapData(const MapData&)’:
/home/dominik/catkin_hydro/src/rtabmap/src/rviz/MapCloudDisplay.cpp:256:135: 
error: too few arguments to function ‘pcl::PointCloud<pcl::PointXYZRGB>::Ptr 
rtabmap::util3d::cloudFromDepthRGB(const cv::Mat&, const cv::Mat&, float, 
float, float, float, int)’
/home/dominik/catkin_hydro/devel/lib/rtabmap-0.7/../../include/rtabmap-0.7/rtabm
ap/core/util3d.h:157:52: note: declared here
make[2]: *** 
[rtabmap/CMakeFiles/rtabmap_ros.dir/src/rviz/MapCloudDisplay.cpp.o] Error 1
make[1]: *** [rtabmap/CMakeFiles/rtabmap_ros.dir/all] Error 2
make: *** [all] Error 2


What version of the product are you using? On what operating system?

ROS Hydro, Ubuntu 12.04

Please provide any additional information below.

Was working correctly until I updated rtabmap library to newest revision

Original issue reported on code.google.com by domi...@almende.org on 28 Jul 2014 at 10:38

Copied from original issue: matlabbe/rtabmap#12

[matlabbe]

From @GoogleCodeExporter on May 26, 2015 18:56

Thx, I'm working on it! (this is because of the new camera calibration feature, 
I use now fx,fy,cx,cy parameters instead of only depthConstant (1/fy))

Original comment by matla...@gmail.com on 28 Jul 2014 at 1:08

• Changed state: Started

[matlabbe]

From @GoogleCodeExporter on May 26, 2015 18:56

No worries, just thought of mentioning it so that it's not forgotten. By the 
way, really great software!! It works like a charm, amazing :)

Original comment by domi...@almende.org on 28 Jul 2014 at 2:09

[matlabbe]

From @GoogleCodeExporter on May 26, 2015 18:56

should be ok from r1617

Original comment by matla...@gmail.com on 28 Jul 2014 at 2:27

• Changed state: Fixed