-
Notifications
You must be signed in to change notification settings - Fork 48
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
Yaw_tolerance of intermediate nodes for edges with move_base type actions #366
Comments
Yes I will work on this, it seems hard to fit within the current framewrok, but let me give it a think |
I'm not sure I think this to be a good idea. It's in some violation to the concepts. If one wants to go to a specific pose, then one should tell topo nav to go to that point. The problem described in https://github.com/LCAS/RASberry/issues/157 should IMO not be solved on the topological navigation level. Using a node orientation to make navigation more robust is a bad hack. We must work towards a navigation system that can enter row, irrespective of the orientation it had before. |
I was thinking on having specific transition nodes (via rosparam) for which going to the fixed position is mandatory, this kind of rule exist when there is a transition between |
I can be handy, but all I wanted to say is that we shouldn't use it to bad-fix any navigation problems. |
I understand that, what I was wondering is more "conceptual" and was looking for your opinion. |
So, conceptually, you want to indicate if the outgoing edge requires the specific orientation to be adhered to, right. So, it should be an optional flag on the outgoing edges? |
At the moment, for intermediate nodes (and edges with move_base actions), the yaw_tolerance is ignored.
However, if only move_base actions are used for all edges, it might still be ideal for the robot to satisfy the yaw_tolerance for some specific intermediate nodes. A possible solution would be to add another attribute to the node definition which determines whether yaw_tolerance should be satisfied for an intermediate node of a route.
The text was updated successfully, but these errors were encountered: