Skip to content
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.

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

Modifications to isStuckCondition node #490

Merged
merged 2 commits into from
Mar 6, 2019

Conversation

orduno
Copy link
Contributor

@orduno orduno commented Dec 13, 2018

This PR should be merged after the new version of the BT library, which incorporates the onInit() method, has been added to ROS

Basic Info

Addressing issue #469.

Description of contribution in a few bullet points

  • Removed thread from isStuckCondition BT node
  • Pulling ROS node from BT Blackboard

@orduno orduno added 2 - Medium Medium Priority nav2_recovery labels Dec 13, 2018
@orduno orduno added this to the D Turtle Release milestone Dec 13, 2018
@orduno orduno self-assigned this Dec 13, 2018
@orduno orduno requested review from mjeronimo, a user and mhpanah December 13, 2018 23:02
@orduno orduno changed the title Recovery Behaviors: pulling ROS node from BB and removing thread Modifications to isStuckCondition node Dec 13, 2018
@orduno orduno changed the title Modifications to isStuckCondition node [WIP] Modifications to isStuckCondition node Dec 14, 2018
@ghost ghost removed this from the D Turtle Release milestone Jan 29, 2019
@ghost ghost removed 2 - Medium Medium Priority nav2_recovery labels Jan 29, 2019
@orduno orduno changed the title [WIP] Modifications to isStuckCondition node [D-release] Modifications to isStuckCondition node Feb 15, 2019
@orduno orduno changed the title [D-release] Modifications to isStuckCondition node Modifications to isStuckCondition node Mar 6, 2019
@orduno orduno merged commit d43d2d4 into ros-navigation:master Mar 6, 2019
@orduno orduno deleted the issue_469 branch March 6, 2019 00:39
mini-1235 pushed a commit to mini-1235/navigation2 that referenced this pull request Feb 5, 2025
…os-navigation#490)

* Add documentation for wait_for_service_timeout ros-navigation#3960

* Update configuration/packages/configuring-bt-navigator.rst

---------

Co-authored-by: Steve Macenski <[email protected]>
Forsyth-Creations pushed a commit to Forsyth-Creations/navigation2 that referenced this pull request Feb 19, 2025
…ation#490)

* Allow ff_velocity_scale=0 without deprecated warning

* Remove deprecated warning for old ff_velocity_scale param
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