Skip to content

Conversation

@armgits
Copy link
Contributor

@armgits armgits commented Nov 18, 2025


Basic Info

Info Please fill out this column
Ticket(s) this addresses ros-navigation/navigation2#5687
Does this PR contain AI-generated software? No

Description of contribution in a few bullet points

  • Added allow_partial_planning parameter to the Planner Server configuration guide.
  • Added last_reached_index field to output ports for the ComputePathThroughPoses behavior tree node.

@armgits armgits force-pushed the partial-planning-update branch from ad9cff8 to a37c7c0 Compare November 18, 2025 03:58
@armgits
Copy link
Contributor Author

armgits commented Nov 18, 2025

The errors seem to be coming from other files outside I've worked on. Please let me know if I can do something about it.

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.

Add to the migration guide please and otherwise LGTM!

@armgits
Copy link
Contributor Author

armgits commented Nov 18, 2025

Will these changes be added in Kilted to L-turtle migration guide?

@SteveMacenski
Copy link
Member

Yup!

@SteveMacenski
Copy link
Member

(by that I mean you should add it here so its included :-) )

@armgits
Copy link
Contributor Author

armgits commented Nov 18, 2025

I just added that too

@SteveMacenski SteveMacenski merged commit b601dc7 into ros-navigation:master Nov 18, 2025
5 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