* fix invalid memory access

this fix was applied as part of the noetic release of costmap_2d

* uncrustify

* Fix memory leak in nav2_recoveries

* Fix recovery server memory leak (better interface)

* Fix costmap2d memory leak

* Fix nav2_navfn_planner memory leak

* Fix planner server and navfn planner memory leak

* Make all rclcpp::Node::SharedPtr argument passing const

* Fix controller server and DWB plugins memory leak

* Minor fixes

* Fix formatting errors

* Change all plugin interfaces to use weak_ptr intead of shared_ptr to parent rclcpp::Node

* Convert all SharedPtr to WeakPtr

* Check shared_ptr after lock and before dereferencing

* Fix tests declaring parameters real nodes don't

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Fix loadParameterWithDeprecation not getting initial parameter values

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Create sim_time variable before using it

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Line length < 100

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Add missing {

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Linter fixes

Signed-off-by: Shane Loretz <sloretz@openrobotics.org>

* sim_granularity -> time_granularity

Signed-off-by: Shane Loretz <sloretz@openrobotics.org>

* Linter fix

Signed-off-by: Shane Loretz <sloretz@openrobotics.org>

* remove path_updated blackboard variable to check if the path is updated

Signed-off-by: Daisuke Sato <daisukes@cmu.edu>

* remove path_udpated from tests

Signed-off-by: Daisuke Sato <daisukes@cmu.edu>

* make bt_action_node logic clear and fix test failures

Signed-off-by: Daisuke Sato <daisukes@cmu.edu>

* Revert "make bt_action_node logic clear and fix test failures"

This reverts commit bb29d801d504cf5de8c33524d01f76974f76e23a.

Signed-off-by: Daisuke Sato <daisukes@cmu.edu>

Signed-off-by: Daisuke Sato <daisukes@cmu.edu>

* Remove unused achor

* Trigger dockerhub build via cron job

* Rename workflow

* Change timezone
trigger dockerhub build 12 am pst
test nightly job 6am pst

* Clean up dockerhub job refrences

* Add missing executors for python_exec

* Remove unused devel folder

[WIP] removing rviz plugins from test coverage since its not a production run-time component (#1953)

* removing rviz plugins from test coverage since its not a production run-time component

* Update code_coverage_report.bash

The plan recieved is usually in global frame, but
our local costmap is often in odom frame. This
fixes a regression from #1857

To fix cpplint

* Use colcon to check for build failure
instead of canary file

* Simplify if case for shell

* Move extra dockerfiles to docker related folder

* Update Dockerfile location

* Stage refactor

* Use build kit

* Update underlay and overlay build

* Dockerfile tweek

* Rename underlay .repos file

* Remove uneeded key

* Clone matching distro branch by default

* Add optional directive to RUN colcon test

* Add comments

* Skip the use of symlinks

* Roll back to nightly image with all rmws
now that connext is fixed

* Add slam_toolbox to skipped rosdep keys

* Set DEBIAN_FRONTEND env to noninteractive

* Update to tag foxy

* Updated to latest codecov orb

* bump to 0.4.2

* removing nav2_bringup from binaries

Cleanup CircleCI config

https://circleci.com/docs/2.0/configuration-reference/#matrix-requires-version-21

https://circleci.com/docs/2.0/executor-types/#ram-disks

as nav2_system_tests still dominates total test time

to reduce line noise

to simplify retest-until-pass

* remove dead code not used in 10 years from navfn

* ping

* inverting period to rate

* removing debug statement that could never be triggered by a single non-looping action server

* type change

* adding CLI test

* adding tests for more lines to map_server

* fix last test

* adding out of bounds and higher bounds checks

* adding planner tests for plugins working

* add cleanup

* working

* ping

* adding more testing and debug info messages

* adding CLI test

* adding tests for more lines to map_server

* fix last test

* adding out of bounds and higher bounds checks

* adding planner tests for plugins working

* add cleanup

* working

* ping

* range costmap building

* range sensor layer working

* nav2 costmap pass linter and uncrustify tests

* Added back semicolon to range class

* Added docs

* Added angles dependency

* Added BSD license

* Added BSD license

* Made functions inline

* revmoed get_clock

* added input_sensor_type to docs

* Made defualt topic name empty to cause error

* using chorno literal to denote time units

* Added small initial test

* Fixed linter error, line breaks, enum, logger level, and transform_tolerance issue

* fixed segmentation fault in test and added transfrom tolerances to tf_->transform

* Got test to pass

* Added more tests

* removed incorrect parameter declaration

* Improved marking while moving tests and added clearing tests

* removed general clearing test

* changed testing helper import in test

* Improve interface of NavigateToPose

Signed-off-by: ymd-stella <world.applepie@gmail.com>

* Update bt_conversions.hpp (PoseStamped)

* Use full message of PoseStamped

Signed-off-by: ymd-stella <world.applepie@gmail.com>

* Fix lint error

* adding CLI test

* adding a bunch of new coverages for map_server

* fixing #1917 on declare if not declared

* fix API

* adding waypoint follower failure test

* adding voxel, more logging

* reset -> clear

* remove erraneous handling done by prior

* adding a bunch of voxel unit tests

* retrigger