Fix tf timeout of image_rotate #382
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@JWhitleyAStuff LGTM, it just effects initialization of the 1st image run through. @garaemon please rebase this to the melodic-devel branch and we'll merge! |
SteveMacenski
approved these changes
Apr 10, 2019
JWhitleyWork
approved these changes
Apr 10, 2019
* If prev_stamp_ is initialized as 0.0, tf may wait transformation for long duration at the first time. In order to give up the wait in reasonable duration, initialize prev_stamp_ as current time.
garaemon
force-pushed
the
intiialzie-prev-stamp
branch
from
1957c3d
to
36155bb
Compare
|
I've rebased and changed the target branch to melodic. |
|
Thanks for the contribution! |
SteveMacenski
added a commit
that referenced
this pull request
Apr 11, 2019
Backport of #382: Initialize prev_stamp_ as ros::Timw::now()
generush
pushed a commit
to generush/image_pipeline
that referenced
this pull request
May 9, 2019
…tamp Fix tf timeout of image_rotate
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
long duration for the first time. In order to give up the wait in
reasonable duration, initialize prev_stamp_ as the current time.