Adds missing deps (#36)

* Makes ompl planning_adapters argument optional (#27)

This commit makes sure that the `planning_adapters` argument in the `ompl_planning_pipeline.launch.xml` launch file is optional.

* Fixes custom planner ns bug (#30)

This commit fixes a custom planner ns bug that was introduced by the `setup_assistant` (see
moveit/moveit#2842).

* Adds missing package dependencies

This commit adds several missing package dependencies. These package
dependencies were commented to prevent installing gazebo be default. It
was merged to be consistent with the 'moveit_setup_assistant' (see
moveit/moveit#2839).

package.xml

@@ -30,6 +30,10 @@
   <run_depend>tf2_ros</run_depend>
   <run_depend>xacro</run_depend>
   <run_depend>moveit_visual_tools</run_depend>
+  <!-- The next 2 packages are required for the gazebo simulation.
+      We don't include them by default to prevent installing gazebo and all its dependencies. -->
+  <!-- <run_depend>joint_trajectory_controller</run_depend> -->
+  <!-- <run_depend>gazebo_ros_control</run_depend> -->
   <!-- This package is referenced in the warehouse launch files, but does not build out of the box at the moment. Commented the dependency until this works. -->
   <!-- <run_depend>warehouse_ros_mongo</run_depend> -->
   <run_depend>franka_description</run_depend>