Skip to content

Cherry-Pick Adding slow down at target heading to RPP Controller (#5361) #5443

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 12, 2025
Merged

Cherry-Pick Adding slow down at target heading to RPP Controller (#5361) #5443

merged 2 commits into from
Aug 12, 2025

Conversation

@zeerekahmad
Copy link

@zeerekahmad zeerekahmad commented Aug 12, 2025

Description of contribution in a few bullet points

Cherry picked from #5361

  • Adding slow down at target heading to RPP
  • Update test_regulated_pp.cpp

For Maintainers:

  • Check that any new parameters added are updated in docs.nav2.org
  • Check that any significant change is added to the migration guide
  • Check that any new features OR changes to existing behaviors are reflected in the tuning guide
  • Check that any new functions have Doxygen added
  • Check that any new features have test coverage
  • Check that any new plugins is added to the plugins page
  • If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists
  • Should this be backported to current distributions? If so, tag with backport-*.
@SteveMacenski @zeerekahmad
Adding slow down at target heading to RPP Controller (ros-navigation#…
4ff1fbd 
…5361)

* Adding slow down at target heading to RPP

Signed-off-by: SteveMacenski <stevenmacenski@gmail.com>

* Update test_regulated_pp.cpp

Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>

---------

Signed-off-by: SteveMacenski <stevenmacenski@gmail.com>
Signed-off-by: Steve Macenski <stevenmacenski@gmail.com>
@mergify
Copy link
Contributor

mergify bot commented Aug 12, 2025

@zeerekahmad, all pull requests must be targeted towards the main development branch.
Once merged into main, it is possible to backport to @humble, but it must be in main
to have these changes reflected into new distributions.
@SteveMacenski
Copy link
Member

SteveMacenski commented Aug 12, 2025

This does not compile 🙃
@SteveMacenski SteveMacenski merged commit 476d9e8 into ros-navigation:humble Aug 12, 2025
2 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

2 participants

@zeerekahmad @SteveMacenski