Have to use position-independent code (-fPIC) for code that's in shared
libraries.  Also, had to make other non-node libraries SHARED as well
since they could be used by nodes in node libraries and this code would
need to be position independent as well.

* Enable colcon-test in travis for passing packages

* Use colcon test-result

* Update full list of failing tests

This reverts commit b65d0938.

* Changes for latest TB3 models

* Turning TF bush into TF tree.

Adding a fix to remove the 'hook' artifact at the end of the global plan.

Print a message that a function is skipped if a checkpoint file exists.

* Move code that listens to rviz goal_pose message into the NavigateToPoseTask base class
* Fix merge issue and address lint issues
* Remove redundant #include

renamed voxel grid namespace, fixed compiler issues

remove commented out unneccesary code

remove map_msgs package

set dynamic parameters on initialization

adding comment

renable dwbplugin tests

base include in CMakaeLists and first stab port of obstacle layer

continued porting obstacle layer changes

resolving everything for next person except stringstream + message filters issues

add back current amcl changes

add back current dockerfile changes

uncomment arguments in function

* Enable publishing the costmap for debug purposes.

* Changing costmap publishing loop to use std::chrono time

The costmap update loop is using rclcpp::Rate which is always
std::chrono time. The publish cycle should also use that, otherwise
there can be problems getting data published when switching to
sim_time.

* Added an execution timer helper class.

This hides some of the std::chrono manipulation to convert a
duration to a numeric number of seconds.

Also helps to better express why we are measuring time.

* Switching costmap publishing timer to ROS time.

* Check for a switch to sim time and update last_publish accordingly

* Removing unneeded header

* Adding a missed file.

* Getting started; not working yet
* Update the unit test
* Fix issue #44 to modify TaskServer to accept a ROS node rather than be a ROS node

Updated the TaskClient and TaskServer templates to accept a rclcpp::Node::SharedPtr
reference. Also updated the Robot class to do the same. Also had to then change
the execute method to be set on the TaskServer rather than have derived classes
override the method. Now, it is more like the SimpleActionServer from ROS1,
which sets us up a bit better for the port to Actions when ready on ROS2.

* Rename various execute methods to be specific now that they are not overrides of a base class virtual method
* Costmap service name should just be GetCostmap