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

Update Navigation module #65

Open
kumarkrishna opened this issue May 12, 2014 · 3 comments
Open

Update Navigation module #65

kumarkrishna opened this issue May 12, 2014 · 3 comments

Comments

@kumarkrishna
Copy link
Contributor

TODO List:

  • Remove dependency from Utils
    -It depends on Utils only because of State message type. I think that we should have a Utils package for things like this which will be common to many modules.
  • Port rest of the code from eklavya_navigation
    -Just like FollowNose there are other strategies which have to be ported from the old package.
@kumarkrishna kumarkrishna added this to the Basic Test milestone May 12, 2014
@dragonfly91
Copy link
Contributor

Just one thing: all the strategies are now implemented in the target_selector node. So, port all of them there. Also, #66 would solve this.

@dragonfly91
Copy link
Contributor

any updates on this, @abinashmeher999 ?

@abinashmeher999
Copy link
Member

After refactoring, we will be no more using the Navigator package. Everything in the package has been ported to strategy_planner and the navigators.

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

3 participants