add a wait to give TF time to be available#636
Conversation
|
note, it was successful on the second run. This was the error when it fails: |
|
@bkanator the path tolerance violation is a different issue, probably because Mujoco is being slow - it can't track the path below the configured tolerance |
|
I think this is a timing thing? but, the wait should be moved up. ----- below from claude --- WaitForDuration fix from PR #636 that fix is scoped to inside the Parallel block (for TF propagation of the static frame). But the failure is happening at the very first node: Move to Waypoint → Home, which runs before any of that. Root cause: This is a classic robot state initialization timing issue in MuJoCo sim. On first The WaitForDuration added in PR #636 helps with the TF for the Cartesian path, but it's placed Two options for fixing this: Option A — Add a WaitForDuration at the top of the sequence (simple, reliable): Option B — Wrap Move to Waypoint in RetryUntilSuccessful (cleaner UX, self-healing): Option A is simpler and more predictable. Option B is more robust but hides the latency. I'd lean Worth noting this affects all objectives that start with a Move to Waypoint in kinova_sim — it
|
|
Bumping to 9.3.1. |
|
@bkanator I think we may be conflating two different issues here. The path tolerance during "Move To Waypoint" is a different issue than can happen sometimes in systems where Mujoco is being slow. The solution to that is different - probably just increase the allowed path tolerance in the controller config - but we can do that in a different PR if you keep seeing it |
Yes! multi-tasking is not working well for me. In my brain I was thinking, 'test this to see if it fails', you are correct it is a different issue. |


In
kinova_sim,Write PickNik, add aWaitForDurationafter TF starts getting published, before starting the other parallel thread, to give some time for the TF to be broadcastedCloses https://github.com/PickNikRobotics/moveit_pro/issues/19123