fixed. I usually do this, not sure why I slipped up here.


On Thu, May 15, 2014 at 11:51 AM, Ioan A Sucan <notifications@github.com>wrote:

> In src/tools/moveit_config_data.cpp:
>
> > @@ -296,18 +296,87 @@ bool MoveItConfigData::outputOMPLPlanningYAML( const std::string& file_path )
> >
> >    emitter << YAML::Value << YAML::BeginMap;
> >
> > -  // Add Planners
> > -  static const std::string planners[] =
> > -    {"SBL", "EST", "LBKPIECE", "BKPIECE", "KPIECE", "RRT", "RRTConnect", "RRTstar", "TRRT", "PRM", "PRMstar" };
> > -
> > +  std::vector<OMPLPlannerDescription> planner_des;
> > +
> > +  OMPLPlannerDescription SBL("SBL","geometric");
> > +  SBL.addParameter("range","0.0","Max motion added to tree. ==> maxDistance_ default: 0.0, if 0.0, set on setup()");
>
> Please add a space after every ,
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ros-planning/moveit_setup_assistant/pull/85/files#r12700057>
> .
>