Skip to content

Following#787

Merged
SteveMacenski merged 6 commits intoros-navigation:masterfrom
grupo-avispa:following
Oct 6, 2025
Merged

Following#787
SteveMacenski merged 6 commits intoros-navigation:masterfrom
grupo-avispa:following

Conversation

@ajtudela
Copy link
Copy Markdown
Contributor


Basic Info

Info Please fill out this column
Ticket(s) this addresses https://github.com/ros-navigation/navigation2/pull/5565
Does this PR contain AI-generated software? (No; Yes, and it is marked inline in the code)

Description of contribution in a few bullet points

  • Updated the documentation for the Following server.

Comment thread configuration/packages/configuring-following-server.rst Outdated
Comment thread configuration/packages/configuring-following-server.rst Outdated
Comment thread configuration/packages/configuring-following-server.rst
Comment thread tutorials/docs/navigation2_dynamic_point_following.rst Outdated
</div>
</h1>


Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A really big line break / space here between these to distinguish them better as 2 different techniques would be good

Copy link
Copy Markdown
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.

One last thing: This is worth mentioning in the migration guide with a link to the tutorial / configuration guide

Copy link
Copy Markdown
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.

Want to rebase to pull in the index page changes to insert this? Will merge then :-)

Comment thread configuration/index.rst Outdated
packages/configuring-controller-server.rst
packages/configuring-costmaps.rst
packages/configuring-docking-server.rst
packages/configuring-following-server.rst
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can this be moved to 'Other'? While useful, this is not something 'core' for some systems

@SteveMacenski
Copy link
Copy Markdown
Member

Oh also add your 2x new functions to the Simple Commander API page so folks know to use them!

ajtudela and others added 5 commits October 6, 2025 09:24
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
Signed-off-by: Alberto Tudela <ajtudela@gmail.com>
@SteveMacenski SteveMacenski merged commit 514e85b into ros-navigation:master Oct 6, 2025
5 checks passed
@ajtudela ajtudela deleted the following branch October 7, 2025 09:20
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