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

[segbot_logical_translator] should check whether costmap has been updated whether door/no-door version. #28

Closed
piyushk opened this issue Mar 26, 2015 · 0 comments
Assignees
Milestone

Comments

@piyushk
Copy link
Member

piyushk commented Mar 26, 2015

Right now whenever segbot_logical_translator updates the map being used by navigation, it waits 1 second arbitrarily after publishing the map after which it assumes the map has been changed. This is bad for 2 reasons:

  1. Under greater computational load where the map update process can take more than 1 second - ex. Some of the observations returned after approaching a door may be wrong as move_base may not have updated to the map where doors are not marked as obstacles.
  2. The general case where the update is really quick (<< 1 second) - We wait unnecessarily after the action is complete.

A better way of doing so would be to read the map_load_time of the costmap (hoping that it is set to the time the actual map was loaded in).

@piyushk piyushk self-assigned this Mar 26, 2015
@piyushk piyushk added this to the 0.3.1 milestone Mar 26, 2015
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