moveit / moveit_core

THIS REPO HAS MOVED TO https://github.com/ros-planning/moveit
31 stars 76 forks source link

Add time factor to iterative_time_parametrization #224

Closed skohlbr closed 9 years ago

skohlbr commented 9 years ago

See https://github.com/ros-planning/moveit_ros/pull/547

mikeferguson commented 9 years ago

This PR seems fine if the others are good. Needs to be merged first.

sachinchitta commented 9 years ago

Can you call this something like: max_velocity_scaling_factor since it only changes the max velocity and does not affect the accelerations? "time_factor" is too general to describe what this does.

isucan commented 9 years ago

looks good to me

mikeferguson commented 9 years ago

Whoops, I thought this failed because the moveit_msgs PR had not been merged... however, it appears that the farm just hiccup'd -- going to kick it to run again.

mikeferguson commented 9 years ago

Ok, this builds fine...