moveit2 icon indicating copy to clipboard operation
moveit2 copied to clipboard

Add a "target_pose_lookahead_time" parameter

Open AndyZe opened this issue 3 years ago • 5 comments

This work was started in #886. Closes #696.

The new parameter helps account for latency in the control loop. This can compensate for various issues or delays in the control loop that are hard to quantify, including:

  • joint_states publication rate
  • ROS message transmission. There are usually 2 message hops when Servo is used
  • network latency
  • a little bit of processing time
  • whatever happens in the ros_control interface
  • low-level controller ramp up/ramp down time

AndyZe avatar Jan 13 '22 17:01 AndyZe

@livanov93

AndyZe avatar Jan 13 '22 17:01 AndyZe

This pull request is in conflict. Could you fix it @AndyZe?

mergify[bot] avatar Apr 12 '22 15:04 mergify[bot]

@AndyZe what is the status of this?

henningkayser avatar Aug 10 '22 19:08 henningkayser

It could still be useful. It's stuck somewhere between "potentially useful" and "useful enough that I want to merge it now".

AndyZe avatar Aug 10 '22 19:08 AndyZe

This pull request is in conflict. Could you fix it @AndyZe?

mergify[bot] avatar Sep 27 '22 22:09 mergify[bot]

@AndyZe if you rebase this and it passes CI, I'll approve it. What do you think?

tylerjw avatar Nov 17 '22 22:11 tylerjw

I'll just close this since it's not clear if it would be useful. Let's remember this in the future in case Servo is laggy, though

AndyZe avatar Nov 18 '22 14:11 AndyZe