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

use less strict parameters for ground filtering #29

Closed
wants to merge 1 commit into from

Conversation

v4hn
Copy link
Member

@v4hn v4hn commented Apr 7, 2017

In our setup (an off-the-shelf PR2 with respect to these parts) the
previous parameters were too tight:
In many situations, especially when an obstacle split the ground-part
of a single laser-scan into two segments, one of these segments is not
picked up by the line-RANSAC with the old parameters.
Thus it ends up as an obstacle obstructing the local costmap of the
navigation.
With indigo, nobody else noticed this up to now because of a broken
filter-chain before that(see #27).

In our setup (an off-the-shelf PR2 with respect to these parts) the
previous parameters were too tight:
In many situations, especially when an obstacle split the ground-part
of a single laser-scan into two segments, one of these segments is not
picked up by the line-RANSAC with the old parameters.
Thus it ends up as an obstacle obstructing the local costmap of the
navigation.
With indigo, nobody else noticed this up to now because of a broken
filter-chain before that(see PR2#27).
@v4hn
Copy link
Member Author

v4hn commented Apr 26, 2018

Note that even though this improves the floor detection, it is still not very good and adds artifacts to the floor.
We might work on this in the future, but maybe it would be best to disable this completely (after fixing it as proposed int #27) to have a somewhat working move_base navigation.

@k-okada
Copy link
Contributor

k-okada commented Apr 26, 2018

closes via #35

@k-okada k-okada closed this Apr 26, 2018
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.

2 participants