loam_velodyne icon indicating copy to clipboard operation
loam_velodyne copied to clipboard

/LaserMapping.cpp:149:28: error: ‘downSizeFilterMap’ was not declared in this scope

Open elahea2020 opened this issue 5 years ago • 6 comments

Hi, When I try to make the file I get this error. In member function ‘virtual bool loam::LaserMapping::setup(ros::NodeHandle&, ros::NodeHandle&)’: catkin_ws/src/loam_velodyne/src/lib/LaserMapping.cpp:149:28: error: ‘downSizeFilterMap’ was not declared in this scope downSizeFilterMap().setLeafSize(fParam, fParam, fParam);

Which was not the case before the PR. I looked at the header and the parent class and seems like this function has never been defined.

Would appreciate it if someone could fix this bug.

Thanks

elahea2020 avatar Aug 22 '18 22:08 elahea2020

Please put a link to the offending line in the commit you are using. Also what build tools are you using?

adishavit avatar Aug 23 '18 04:08 adishavit

Hi,

I have the same problem. Here is the link to the line: LaserMapping.cpp:149:28

Thanks

claydergc avatar Aug 23 '18 05:08 claydergc

I have the same problem and this is a new error that has not been seen before.

nature1949 avatar Aug 23 '18 08:08 nature1949

I got the same question, after compiled I got /home/tiffer/test_catkin/src/loam_velodyne-master/src/lib/LaserMapping.cpp:149:28: error: ‘downSizeFilterMap’ was not declared in this scope downSizeFilterMap().setLeafSize(fParam, fParam, fParam); ^ [8/18] Building CXX object src/lib/CMa...Files/loam.dir/BasicLaserMapping.cpp.o ninja: build stopped: subcommand failed. <== Failed to process package 'loam_velodyne': Command '['/home/tiffer/test_catkin/install_isolated/env.sh', 'ninja', '-j8', '-l8']' returned non-zero exit status 1 and for catkin_make also got false just like elahea2020. downSizeFilterMap() Not Define

TifferPelode avatar Aug 23 '18 10:08 TifferPelode

I'll look into it asap.

adishavit avatar Aug 23 '18 12:08 adishavit

You just need to remove that code. The parameter isn't used anywhere in the code. I made a new PR #84 with the fix.

adishavit avatar Aug 23 '18 14:08 adishavit