Skip to content

Conversation

@adivardi
Copy link
Contributor

Update migration guide for ros-navigation/navigation2#5466

@adivardi
Copy link
Contributor Author

Should I also update the BT navigator page? Is it also referring to rolling?

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.

Should I also update the BT navigator page?

Yes please!

@adivardi
Copy link
Contributor Author

I also added the params for the opennav_coverage_navigator, I hope that is OK. The PR changing the coverage navigator is ready but not open yet, just waiting for migration PR to be merged

Signed-off-by: Adi Vardi <[email protected]>
Signed-off-by: Adi Vardi <[email protected]>
Signed-off-by: Adi Vardi <[email protected]>
@SteveMacenski SteveMacenski merged commit 04943e5 into ros-navigation:master Aug 26, 2025
4 checks passed
@adivardi adivardi deleted the av/bb_id branch August 27, 2025 06:37
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