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

Feature/invalid scans1.1.4 #8

Merged
merged 1 commit into from Apr 23, 2016
Merged

Feature/invalid scans1.1.4 #8

merged 1 commit into from Apr 23, 2016

Conversation

ghost
Copy link

@ghost ghost commented Mar 10, 2016

I adapted open_karto to work with the infinite readings from the Robopeak lidar driver on an Odroid XU4 (Exynos5422 Cortex™-A15 2Ghz and Cortex™-A7 Octa core CPUs, probably an ARM issue). The problem is that on the Odroid this infinite readings mess up the scan matching.
Without the patch:
ros-slam_orig-karto_odroid
With the patch:
ros-slam_patched-karto_odroid

On a generic pc (Intel Core i5) this doesn't happen:
Without patch:
ros-slam_orig-karto

Is it right that the scan matcher doesn't consider min and max range?
If you need more information ask and I will be happy to provide them.
I can provide also a rosbag file with laserscan data and odometry to reproduce the problem if you want.

@ghost
Copy link
Author

ghost commented Mar 10, 2016

Just by the way, the graph is in wrong location because of a misconfigured frame_id.

@mikeferguson
Copy link
Member

@rctoris can you take a look at this

@rctoris
Copy link
Contributor

rctoris commented Mar 11, 2016

Looks good!

@mikeferguson
Copy link
Member

@mig-em can you squash these commits into a single one so it is easier for future users/maintainers to see that these changes are related. Thanks!

@ghost
Copy link
Author

ghost commented Mar 14, 2016

I squashed them.

@mikeferguson mikeferguson merged commit b763880 into ros-perception:indigo-devel Apr 23, 2016
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

Successfully merging this pull request may close these issues.

None yet

3 participants