Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As pointed out in #350 (comment), the race condition in
waitForCurrentState()
was not fully resolved yet.The reason was that only the first incoming
joint_state
update triggered aPlanningScene
update, while later updates were throttled. To overcome this issue, we should explicitly trigger a scene update inwaitForCurrentState()
, if there were (further) updates.As verified by @mmoerdijk, this PR finally resolves the issue.
The unittest so far only involved fake controllers that didn't revealed the timing issue.