Skip to content

Conversation

@JatinPatil2003
Copy link
Contributor

@JatinPatil2003 JatinPatil2003 commented Jun 16, 2024


Basic Info

Info Please fill out this column
Ticket(s) this addresses (add tickets here #4238 )
Primary OS tested on (Ubuntu, MacOS, Windows)
Robotic platform tested on (Steve's Robot, gazebo simulation of Tally, hardware turtlebot)
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


Future work that may be required in bullet points

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
Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
@JatinPatil2003 JatinPatil2003 force-pushed the RPP_deceleration_4238 branch from 2c173be to 6a4a4d5 Compare June 16, 2024 08:07
Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
Copy link
Member

@padhupradheep padhupradheep left a comment

Choose a reason for hiding this comment

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

LGTM! Tested it as well!

Great job @JatinPatil2003 Thanks for the contribution

@padhupradheep
Copy link
Member

@JatinPatil2003 Could you please also give in a PR for the documentation? Then we can get this inn..

Thanks!

@JatinPatil2003
Copy link
Contributor Author

I have opened a PR for documentation ros-navigation/docs.nav2.org/pull/567

@SteveMacenski SteveMacenski merged commit 8f795a2 into ros-navigation:main Jun 18, 2024
@SteveMacenski
Copy link
Member

SteveMacenski commented Jun 18, 2024

Thanks @JatinPatil2003 and @padhupradheep - great contributions!

@padhupradheep
Copy link
Member

was about to press merge @SteveMacenski :-D

Marc-Morcos pushed a commit to Marc-Morcos/navigation2 that referenced this pull request Jul 4, 2024
* Added RPP use_cancel_deceleration parameter

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

* Fixed Linting code style

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

---------

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
Manos-G pushed a commit to Manos-G/navigation2 that referenced this pull request Aug 1, 2024
* Added RPP use_cancel_deceleration parameter

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

* Fixed Linting code style

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

---------

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
masf7g pushed a commit to quasi-robotics/navigation2 that referenced this pull request Oct 23, 2024
* Added RPP use_cancel_deceleration parameter

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

* Fixed Linting code style

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

---------

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
stevedanomodolor pushed a commit to stevedanomodolor/navigation2 that referenced this pull request Apr 29, 2025
* Added RPP use_cancel_deceleration parameter

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

* Fixed Linting code style

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>

---------

Signed-off-by: Jatin Patil <jatinpatil2003@gmail.com>
Signed-off-by: stevedanomodolor <stevedan.o.omodolor@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants