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

[VDA5050] Navigate through nodes #47

Open
wants to merge 23 commits into
base: humble-devel
Choose a base branch
from

Conversation

bobbleballs
Copy link
Contributor

Addresses #44
The connector looks through the order that is accepted. If there are multiple released nodes that can be traversed without stopping (No HARD or SOFT actions) then the NavigateThroughNodes actions will be used instead.
For single nodes then the standard NavigateToNode action will be used. There is a parameter that will turn navigate through nodes, it is off by default to keep compatibility with the previous version.
A limitation is that an update to the order, i.e STITCH or UPDATE will not be applied straight away so there will be a short pause in-between whilst the next navigation is loaded after the current base is finished.

I have some examples that I will collate together into the Turtlebot3 Adaptor in vda5050_adapter_examples.

ben.holden and others added 23 commits July 5, 2023 17:49
…e it doesn't seem to be working correctly.
…ess next edge timer kicks off at the same time as the navigation callback causing a double navigation.
@miguelgarcia
Copy link
Member

Hey @bobbleballs since adding the new behavior makes the connector a little more complex, do you think there is a real advantage is using NavigateThroughNodes vs multiple calls to NavigateToNode ? How does this improve the robot behavior ?

Thanks!

@bobbleballs
Copy link
Contributor Author

Hey @bobbleballs since adding the new behavior makes the connector a little more complex, do you think there is a real advantage is using NavigateThroughNodes vs multiple calls to NavigateToNode ? How does this improve the robot behavior ?

Thanks!

Hi @miguelgarcia NavigateToNode requires the robot to come to a complete stop at each node call. When you are running robots in a situation where you have to traverse many nodes to navigate through a building this actually adds a lot of time to the navigation.
The method that I've used here allows for multiple navigation nodes (i.e. where there are no actions to be performed) to be combined in to a follow via waypoint fashion, akin to the navigate through poses actions in Nav2.
The robot is able to traverse multiple nodes without stopping this way which speeds up navigation a lot.

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.

3 participants