Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
elahea2020 opened this issue Aug 22, 2018 · 6 comments

Comments

@elahea2020
Copy link

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

@adishavit
Copy link
Contributor

adishavit commented Aug 23, 2018 via email

@claydergc
Copy link

Hi,

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

Thanks

@nature1949
Copy link

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

@TifferPelode
Copy link

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

@adishavit
Copy link
Contributor

I'll look into it asap.

@adishavit
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants