Unify parameters names for local planner (trivial change)

[ros-planning]

Unify parameters names for local planner (trivial change)

[felix-kolbe]

+1
Having such differences there is really bad, fiddling with the navigation configuration is puzzling itself. Could it be you changed the wiki page for every distro? A note (or distro-branching) would be good to prevent pre-hydro users hit their computers or robots.

[DLu]

I made a change for this in my layered-groovy code. I can port to Hydro if it all looks good.

[mikeferguson]

A little late to do this to either hydro or indigo, propose we do this in Jade.

[mikeferguson]

@corot -- any chance you could put together a Pull Request to fulfill this in Jade?

[corot]

mmm.... I'm horribly busy these days, so I cannot promise anything.

[corot]

Another worse case of duplicated parameters: base local planner uses both max/min_vel_theta and max/min_vel_x. So you must set both!
👍 to keep only max/min_vel_x. Would be great to do it before Kinetic release

[mikeferguson]

Attacking this for Melodic

[mikeferguson]

The defaults for path_distance_bias and pdist_scale in the two dynamic reconfigurations are vastly different (32 vs 0.6) -- are these really "the same thing"?

[DLu]

Closed in #725