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

NavFn now plans paths through places it should not. #28

Closed
hershwg opened this issue Apr 5, 2013 · 2 comments
Closed

NavFn now plans paths through places it should not. #28

hershwg opened this issue Apr 5, 2013 · 2 comments

Comments

@hershwg
Copy link
Contributor

hershwg commented Apr 5, 2013

This is probably an unintended consequence of my fix for #26, not sure. It's possible this is caused by a bug in costmap_2d, but regardless it is definitely a bug.

path-planning-fail

@hershwg
Copy link
Contributor Author

hershwg commented Apr 9, 2013

Just caught this happening while saving costmaps to files. This screenshot shows the navtest binary (maroon and with path, small on left) and a 4x expansion of the area in question in the gimp image program with the grayscale curve tweaked to highlight differences right at the top end of the range. The white pixels there are 255, next down are 254, and the inflated areas next down are 253. The 255 values indicate "unknown" which you can see is where the path is being planned through in the navtest image.

One thing to do is just set allow_unknown to false in navfn. However it would be even better for the unknown values to be inflated while still in the costmap.
nav-bug-details

@hershwg
Copy link
Contributor Author

hershwg commented Apr 9, 2013

It's not clear to me yet why these "unknown" values from the obstacle costmap are overwriting whatever is in the static map, but regardless it is not the fault of Navfn.

To fix this, just set rosparam /move_base/NavfnROS/allow_unknown to false, and Navfn will stop planning paths through the unknown areas.

@hershwg hershwg closed this as completed Apr 9, 2013
nlimpert pushed a commit to nlimpert/navigation that referenced this issue Feb 25, 2020
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

1 participant