Skip to content

Fix broken URLs in Node Hybrid#4973

Merged
SteveMacenski merged 1 commit intoros-navigation:mainfrom
mattbooker:fix-broken-urls-in-node-hybrid
Mar 10, 2025
Merged

Fix broken URLs in Node Hybrid#4973
SteveMacenski merged 1 commit intoros-navigation:mainfrom
mattbooker:fix-broken-urls-in-node-hybrid

Conversation

@mattbooker
Copy link
Copy Markdown
Contributor


Basic Info

Info Please fill out this column
Ticket(s) this addresses N/A
Primary OS tested on Untested - changes only made to comments
Robotic platform tested on Untested - changes only made to comments
Does this PR contain AI generated software? No

Description of contribution in a few bullet points

  • Fixed URL inside node hybrid to point to correct website

Description of documentation updates required from your changes

None

Description of how this change was tested

Not tested. Change only affects comments


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: mattbooker <[email protected]>
@mattbooker mattbooker force-pushed the fix-broken-urls-in-node-hybrid branch from 3e6473a to 026e141 Compare March 9, 2025 13:58
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
nav2_smac_planner/src/node_hybrid.cpp 90.27% <ø> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
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.

Thanks Matt, long time no talk!

@SteveMacenski SteveMacenski merged commit 8dde70c into ros-navigation:main Mar 10, 2025
SteveMacenski pushed a commit that referenced this pull request Apr 15, 2025
SteveMacenski added a commit that referenced this pull request Apr 15, 2025
* Add short delay before lookup transform in polygon test (#4939)

* Add short delay before lookup transform

Signed-off-by: mini-1235 <[email protected]>

* Fix linting

Signed-off-by: mini-1235 <[email protected]>

---------

Signed-off-by: mini-1235 <[email protected]>

* Populate stamped command message with now timestamp, if velocity timed out. (#4959)

Signed-off-by: Georg Flick <[email protected]>

* Integrate ClearCostmapExceptRegion and ClearCostmapAroundRobot Services into the API (#4962)

* Add ClearCostmapExceptRegion and ClearCostmapAroundRobot services to BasicNavigator API

Signed-off-by: BCKSELFDRIVEWORLD <[email protected]>

* typo ament_flake8

Signed-off-by: BCKSELFDRIVEWORLD <[email protected]>

* type fix ament_flake

Signed-off-by: BCKSELFDRIVEWORLD <[email protected]>

---------

Signed-off-by: BCKSELFDRIVEWORLD <[email protected]>

* Fix urls in node hybrid (#4973)

Signed-off-by: mattbooker <[email protected]>

* Use main function to replace global variable in gtest. (#4978)

Signed-off-by: ChenYing Kuo <[email protected]>

* nav2_behavior_tree: fix input port parsing error in AreErrorCodesPresent (#4986)

The getInput method does not support std::set<uint16_t> parsing. So,
let's replace the type of the input port by std::vector<int> which is
supported, and convert the list to a std::set<uint16_t>.

This commit fixes issue #4985.

Signed-off-by: Dylan De Coeyer <[email protected]>

* Change to goal checker orientation for yaw angle (#4988)

- Fixed discrepancy in goal checker orientation, which was checking for
< tolerance instead of <= tolerance, as all the other limit checks are.
- Reduced tolerance time for the progress checker unit tests to
0.1 seconds, to reduce test runtime from ~17 to ~7 seconds.

Signed-off-by: Rasmus Larsson <[email protected]>

* Declare_parameter_if_not_declared in docking navigator (#5023)

Signed-off-by: Alberto Tudela <[email protected]>

* bumping to 1.3.6 for release

Signed-off-by: Steve Macenski <[email protected]>

---------

Signed-off-by: mini-1235 <[email protected]>
Signed-off-by: Georg Flick <[email protected]>
Signed-off-by: BCKSELFDRIVEWORLD <[email protected]>
Signed-off-by: mattbooker <[email protected]>
Signed-off-by: ChenYing Kuo <[email protected]>
Signed-off-by: Dylan De Coeyer <[email protected]>
Signed-off-by: Rasmus Larsson <[email protected]>
Signed-off-by: Alberto Tudela <[email protected]>
Signed-off-by: Steve Macenski <[email protected]>
Co-authored-by: mini-1235 <[email protected]>
Co-authored-by: Georg Flick <[email protected]>
Co-authored-by: Burak Can Kaya <[email protected]>
Co-authored-by: Matthew Booker <[email protected]>
Co-authored-by: ChenYing Kuo (CY) <[email protected]>
Co-authored-by: DylanDeCoeyer-Quimesis <[email protected]>
Co-authored-by: RasmusLar <[email protected]>
Co-authored-by: Alberto Tudela <[email protected]>
stevedanomodolor pushed a commit to stevedanomodolor/navigation2 that referenced this pull request Apr 29, 2025
Signed-off-by: mattbooker <[email protected]>
Signed-off-by: stevedanomodolor <[email protected]>
SakshayMahna pushed a commit to SakshayMahna/navigation2 that referenced this pull request Jun 8, 2025
Signed-off-by: mattbooker <[email protected]>
Signed-off-by: Sakshay Mahna <[email protected]>
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