Signed-off-by: Pablo Vega <epvega@gmail.com>

* Map Server docs update for Foxy

* Fixes after review

* Add brief description of map_io

* [WIP] map_server refactor and cleanup

nav2_map_server/mapio (map input-optput library):
* Move OccupancyGrid messages saving code from MapSaver::mapCallback()
  to saveMapToFile() function
* Rename and move try_write_map_to_file() MapSaver method
  to tryWriteMapToFile() function
* Move map saving parameters into one SaveParameters struct
* Reorganize map saving parameters verification code from MapSaver
  to new checkSaveParameters() function
* Correct logging for incorrect input cases in checkSaveParameters()
* Copy loadMapFromYaml() method from OccupancyGridLoader
* Move loadMapFromFile() method from OccupancyGridLoader
* Rename and move load_map_yaml() OccupancyGrid method
  to loadMapYaml() function
* Move LoadParameters struct from OccupancyGridLoader

nav2_map_server/map_saver:
* Completely re-work MapSaver node:
 - Switch MapSaver from rclcpp::Node to nav2_utils::LifecycleNode
 - Revise MapSaver node parameters model
 - Add saveMapTop...

Signed-off-by: Shrijit Singh <shrijitsingh99@gmail.com>

The problem appears while addressing zero-length path vector
after planner failed to create a plan.

* Nav2 planner plugin tutorial draft 1

* Reduced gif size

* Updated the config code block

* Remove RRTConnect plugin and add StraightLine plugin.

* Address reviewer's comments

* Add information about planners mapping in Navigation2

* Changed onCycleUpdate to allow preemption (#1622)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Deleted unnecessary wait and corrected style (#1622)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Updated website gifs (#1228)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Created new gifs (#1228)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Corrected gif name (#1228)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Added thread synchronization to KinematicParameters (#1459)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Doxygen fix (#1459)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

A BT port is introduced in case the user wants to change the default action_name
of BtActionNode.

- `BtServiceNode::check_future()` was created, to encapsulate the logic
where a the output of the BtServiceNode is computed.
- Inherited classes can overload this function according to the requirement
of the user

* Proposed refactoring to avoid issues with CoroAction

* correctly haltAllActions (related to #1600)

* not really needed and will be deprecated soon

* Applying changes suggested in the comments of #1606

- fix haltAllActions
- changes method signature on_success()
- reverts the changes made here:
https://github.com/ros-planning/navigation2/pull/1515/files

* fix warnings and errors

* make uncrustify happy?

* Update bt_navigator.cpp

* Update bt_navigator.cpp

* uncrustify fix

Co-authored-by: daf@blue-ocean-robotics.com <Davide Faconti>

* corrected wrong indexing in NavfnPlanner::smoothApproachToGoal function

* resolved uncrustify errors

* corrected the condition inside NavfnPlanner::smoothApproachToGoal to replace last pose of computed path

Co-authored-by: chikmagalore.thilak <chikmagalore.thilak@bshg.com>

* Corrected check to detect collision (#1404)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Changed collision check to detect only lethal and unknown cells (#1404)(#1603)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

the removed code has no effect at all: the status of a node will be its returned value!.

In general, you never set your status to IDLE, unless halted.

* Add eloquent option to build script and made it default

* Add eloquent in the error message

* Remove dashing

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* fixing BT node action processors

* fix crusty code

* remove server timeout

* Included widget to rviz plugins for displaying status of lifecycle managers (#1479)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Updated state machine and InitalThread (#1479)

Changed InitialThread to check the status of each lifecycle manager periodically. Modified state machine to support updating the state of the lifecycle managers.

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Revert "Updated state machine and InitalThread (#1479)"

This reverts commit 3ef5ae4206e24013b3f3c611438b00421ff08d57.

* Revert "Included widget to rviz plugins for displaying status of lifecycle managers (#1479)"

This reverts commit 7befdb3ffd7365faf17120a39caf87a362411a27.

* Added QLabels showing state of lifecycle managers after initialization (#1479)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Corrected typo (#1479)

Signed-off-by: Aitor Miguel Blanco <aitormibl@gmail.com>

* Correc...

It made segfault when running map_saver using option param without value.
When there are wrong args, terminate the process.

Signed-off-by: Koeun Choi <gony0206@gmail.com>

* adding logging for nightly tests

* adding destroy call to action client

* reenable planner test

* try simtime

* adding sim_time in executable line

* try adding a rest

moving xy_goal_tolerance and trans_stopped_velocity param names out of rotatetogoal namespace (#1575)

* fixing website hyperlink for brian wilcox

* BT condition node to check for a valid TF transformation

* fix linter issues

* fix typo

* covering A* case in code coverage

* add a couple tests to see if that improves coverage

* added more videos on roscon talks to the tutorials, similar to creating a costmap layer.

* formatting.

* grammar fixes.

* Some minimal changes from locusrobotics/robot_navigation

- Add publisher checks to publisher.cpp
- Method name change from addGridScore to addCriticVisualization

* Add test for trajectory_utils

- Fix namespace for trajectory_utils

* Address reviewer's comments

* Change if statments

* Minor fixes