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

AMCL doesn't throw error when it receives maps on the correct topic but not matching the global_frame_id #592

Closed
vik748 opened this issue Jul 20, 2017 · 5 comments

Comments

@vik748
Copy link
Contributor

vik748 commented Jul 20, 2017

Currently,
AMCL seems to proceed as if every thing is good if it receives maps on the specified map_topic even if the frame_id on the map doesn't match global_frame_id. In this case AMCL essentially is unable to scan map to the map being published and thus the error on the particles keep growing. A simple fix should be able to warn users that the map being published is not being used by AMCL because of the mismatch.

Cheers,
Vik

@mikeferguson
Copy link
Contributor

Seems reasonable -- pull request for proposed simple fix would be accepted.

@vik748
Copy link
Contributor Author

vik748 commented Jul 28, 2017

@mikeferguson I am very new to github. What is the best way for me to submit the changes? Do I created a new forked repo and then submit that or do I create a new branch in this repo and request that to be merged?

Thanks,
Vik

@mikeferguson
Copy link
Contributor

@vik748 new forked repo (you won't have permissions to create branches on this particular repo). Ideally a feature branch within your repo (so you can easily pull in updates to your fork). Then open a PR against this repo. Thanks!

@vik748
Copy link
Contributor Author

vik748 commented Aug 1, 2017

Pull request #604 submitted.

@mikeferguson
Copy link
Contributor

Fixed in kinetic & lunar

gerkey pushed a commit to codebot/navigation that referenced this issue Jan 19, 2018
johaq pushed a commit to CentralLabFacilities/navigation that referenced this issue Mar 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants