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

Added Overview section to README #4

Merged
merged 2 commits into from
Jun 13, 2018

Conversation

mkhansenbot
Copy link
Collaborator

This overview is based on the current understanding of the existing ROS navigation system and the expected behavior of the new system to be the same. Feedback welcome.

@mikeferguson
Copy link
Contributor

I would point out that currently the system requires "a map, a destination, and a position estimate". While AMCL does maintain localization of the robot based on odometry+laser, it does not localize a robot without some apriori knowledge (the "initial pose").

Given that solving the kidnapped robot problem is still a fairly open research question, I don't think we want to presuppose that we will have such a solution in the new system.

@mkhansenbot
Copy link
Collaborator Author

@mikeferguson - That makes sense, I agree. I updated the README to include "a current pose" as an input.

As for the "Kidnapped Robot" problem, I don't expect this to be solved as part of this project. However I do expect it will be solved in the near future with a combination of positioning technologies, and when it is, this project should work the same as if a user designated an initial pose manually. :)

@mkhansenbot
Copy link
Collaborator Author

Any other comments or can I merge this?

@mkhansenbot mkhansenbot merged commit fa9adfa into ros-navigation:master Jun 13, 2018
@mkhansenbot mkhansenbot deleted the sys_overview branch June 13, 2018 22:42
Marwan99 pushed a commit to Marwan99/navigation2 that referenced this pull request Oct 22, 2021
ghost referenced this pull request in logivations/navigation2 Mar 7, 2022
…ng_branch

Added latest changes from tracking branch
doisyg added a commit to doisyg/navigation2 that referenced this pull request Oct 5, 2023
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

4 participants