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

PoseStampedArray docs #630

Merged

Conversation

tonynajjar
Copy link
Contributor

Copy link
Member

@SteveMacenski SteveMacenski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned in the PR, the config docs for the BT nodes that changed needs to be reflected as well since we give the input/output ports and their types.

Does any of the docs w.r.t. Nav2 BT explanations and walk throughs need to be updated? https://docs.nav2.org/behavior_trees/index.html

migration/Jazzy.rst Outdated Show resolved Hide resolved
migration/Jazzy.rst Outdated Show resolved Hide resolved
tonynajjar and others added 3 commits January 8, 2025 12:29
Co-authored-by: Steve Macenski <[email protected]>
Signed-off-by: Tony Najjar <[email protected]>
Signed-off-by: Tony Najjar <[email protected]>
@tonynajjar
Copy link
Contributor Author

Idk what DCO keeps complaining about, as far as I see all my commits are signed off

@tonynajjar
Copy link
Contributor Author

Does any of the docs w.r.t. Nav2 BT explanations and walk throughs need to be updated?

As far as I see not

@SteveMacenski SteveMacenski merged commit a77b51d into ros-navigation:master Jan 9, 2025
3 of 4 checks passed
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.

2 participants