* Fix SEGV caused by goal_handle_ which is nullptr while result_callback called

* Ignoring the goal result if a new goal request sent

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* removing dashing as EOL

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

* adding galactic to readme table

Fixes #2358

* Fix action syntax

add missing required key

* Fix curl scripting

* Use tokenless codecov upload
by leting codecov script pickup on circleci env
on it own to corroborate identity

* Remove dockerhub cron job from circleci

* Add dockerhub cron job from gh actions

tf2_geometry_msgs.h was replaced by tf2_geometry_msgs.hpp in https://github.com/ros2/geometry2/pull/418.
tf2_sensor_msgs.h was replaced by tf2_sensor_msgs.hpp in https://github.com/ros2/geometry2/pull/416

.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Rename rclcpp parameter exception

The parameter exception name is being changed in https://github.com/ros2/rclcpp/pull/1673



Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Exception is thrown when getting the parameter

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Fix test

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Fix typo

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Update BT nodes to use callback groups / executors to process

implement #2251

* add executor to other BT nodes

* use spin_some in is_battery_low BT condition

* fix merge

* remove spinning thread

* Disable dockerhub workflow for testing

* Disable debug jobs for testing

* Disable extra rmw jobs for testing

* Rename docker images for testing

* Default to cache from main branch

* Use colcon cache

* Install colcon cache

* Fix xargs typo

* List names only

* Checksum overlay ws src after restore and store
to restore ws cache for incremental builds

* Disable test jobs

* Revert checksum update but use underlay src

* Only cache ws build and install folder
to avoide overwritting ws src folder with restore

* Don't remove git files from underlay
so vcs tools can export underlay to checksum

* Save build cache regardless of failure

* cache tests

* Update colcon cache

* No need to remove log folder
as its no longer cached

* Bypass .dockerignore for dockerhub build hooks
to include .git folder in overlay for colcon cache

* Simplify checkout
now that underlay and overload have lockfiles

* Remove overlay src folder for checkout

* Echo env for debug

* Use if checks for empty package sets
as --packages-above with empty args will not skip any package

* Save workspace log to cache

* Set TEST_PACKAGES to empty instead of skipping

* Update colcon cache

* Update colcon cache

* Explicitltly list packages to build
using an environment variable

* Enable debug and nightly jobs

* Test all packages if cache_test is false

* Revert testing fork changes

* Install pip in rolling image

* Rename checksum file to lockfile.txt

* Config formatting

* Simplify testing

* Trigger dockerhub build when ros2 has updates

* Apt upgrade in dockerfile
to avoid dockerhub build trigger loops
when latest ros docker image is outdated
as compared to latest sync ros2 list repo

* Update Dockerfiles and Dockerhub to rolling

* Comment out packages avalable in rolling

* Install RMWs as CI dependencies

* Add action feedback information to nav2 rviz panel

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Reset based on goal status

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Add goal handle resets on goal completeion

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

Use incremental timeouts in spin_until_future_calls during BT execution to handle large server timeouts (#2320)

* Use incremental timeouts in spin_until_future_calls during BT execution

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Fix tests

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Fix tests

* Remove unnecessary timeout_ variable

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Reset goal_sent on terminal conditions

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Rename variables and refactor timeout logic

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Fix timeout logic and add tests

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Updates tests to ensure previous node state does not leak into new cycles

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Change goal_handle_ checks to future_goal_handle_

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* Remove goal handle reset in halt

* Update server_timeout to 20ms and inherit BtServiceNode from BT::ActionNodeBase

Signed-off-by: Sarthak Mittal <sarthakmittal2608@gmail.com>

* goal checker selector

* updating main nav yaml configuration file to support multiple goal checkers

* imised defaiñt gpañ cjecler àra,eter

* changes requested in the PR

* improving selector subscribers for laching behavior

* fixing cpplint uncristify style

* fixing issues

* setting original yaml configuration for the general_goal_checker

* the usage of the goal_checker vector now totally mimic the controller vector

* [Selector Actions] re-enabling back transient local for subscribers and tests

* commit failure on non default avaialable

* Some comments in selector behaviors code and some aesthetic changes

* fixing some linting issues

* minor linting error

Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>

`local` in bash can only be used within a function context.

After the recent Codecov security incident[1] I've been reviewing
codecov usage across ROS repositories.

This repository is fetching the codecov bash uploader without performing
the recommended validation step.

The validation step does not appear to have been widely explained or
publicised and even the official codecov GitHub action was not
validating the script until the recent security incident.

I have made an attempt to validate the bash uploader here.

[1]: https://about.codecov.io/security-update/

* open option to use parent node

* add constructor and clean

Co-authored-by: Simon Honigmann <shonigmann@blueorigin.com>

* remove '/' from node name

* add namespace argument

* use namespace for service client node creation

* ns instead of namespace_

* const

* ns instead of namespace_ in lifecycle_node too

* lint

* Fix for wings artifacts: adding new filters entity to separately update costmaps

* Update nav2_costmap_2d/src/layered_costmap.cpp

Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>

* Fix review items

* Fix/add tests

* Rename costmaps

Co-authored-by: Steve Macenski <stevenmacenski@gmail.com>