ros2 / message_filters

BSD 3-Clause "New" or "Revised" License
76 stars 66 forks source link

Add a simpler aproximate time sync policy: ApproximateEpsilonTime #84

Closed ivanpauno closed 2 years ago

ivanpauno commented 2 years ago

This adds another approximate time policy. The difference is that it doesn't use a relatively complex algorithm to check if the topics are in sync.

It works like ExactTime, but accepts timestamps being within an "epsilon" tolerance.

Todo:

(I have done some manual testing using this in stereo image proc, it seems to work fine)

ivanpauno commented 2 years ago
ivanpauno commented 2 years ago

@gbiggs could you take another look?

ivanpauno commented 2 years ago

Thanks for the reviews @jacobperron @gbiggs !!!

ZhenshengLee commented 1 year ago

It works like ExactTime, but accepts timestamps being within an "epsilon" tolerance.

According to our test, the epsilon version causes less latency to get a good enough sync results in a 8-cameras image pipeline.

Thanks!