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

Changes to the original loam_velodyne code #1

Open
YoshuaNava opened this issue Aug 31, 2018 · 2 comments
Open

Changes to the original loam_velodyne code #1

YoshuaNava opened this issue Aug 31, 2018 · 2 comments

Comments

@YoshuaNava
Copy link

Hey,
I'm Yoshua, robotics engineer working in Zürich, Switzerland but originally from Venezuela. Nice to meet you!

I'm interested in the work you're doing. I've been doing research on LiDAR SLAM and loop closure detection, with the purpose of further increasing the accuracy of LOAM, obtaining both globally consistent and locally accurate maps. As you, I have observed that loam_velodyne from laboshinl doesn't give highly accurate results as the ones claimed by Zhang et al in their papers.

I wanted to ask you, first of all, what kind of changes you did to the LOAM code, and if you took any of the existing LOAM forks as a starting point (for example: laboshinl's fork, lego-loam, the original version from zhang, martin-velas, etc). I would be interested in collaborating to make the open source version of LOAM as accurate as possible.

Thank you.

@claydergc
Copy link
Owner

claydergc commented Aug 31, 2018 via email

@jianrui1
Copy link

@claydergc I have done as you code. But a problem appeared.As follow:
terminate called after throwing an instance of 'std::logic_error'
what(): basic_string::_M_construct null not valid
why?can you tell me the reason and help me out?
Thank you so much

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

3 participants