Skip to content

Commit

Permalink
revert min-fps change from 18a497e
Browse files Browse the repository at this point in the history
  • Loading branch information
maloel committed Oct 26, 2023
1 parent 1ac1a2a commit 7a7fed0
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/sync.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -748,15 +748,15 @@ namespace librealsense
// The threshold is a function of the FPS, but note we can't keep more frames than
// our queue size and per-stream archive size allow.
auto const waiting_fps = get_fps( waiting_to_be_released );
auto const missing_fps = next_expected.fps;
auto const min_fps = std::min( waiting_fps, missing_fps );
//auto const missing_fps = next_expected.fps;
//auto const min_fps = std::min( waiting_fps, missing_fps );

rs2_time_t now = last_arrived.timestamp;
if( now > next_expected.value )
{
// Wait for the missing stream frame to arrive -- up to a cutout: anything more and we
// let the frameset be ready without it...
auto gap = 1000. / min_fps;
auto gap = 1000. / waiting_fps;
// NOTE: the threshold is a function of the gap; the bigger it is, the more latency
// between the streams we're willing to live with. Each gap is a frame so we are limited
// by the number of frames we're willing to keep (which is our queue limit)
Expand All @@ -780,7 +780,7 @@ namespace librealsense
return true;
}

return ! are_equivalent( waiting_to_be_released->get_frame_timestamp(), next_expected.value, min_fps );
return ! are_equivalent( waiting_to_be_released->get_frame_timestamp(), next_expected.value, waiting_fps );
}

bool timestamp_composite_matcher::are_equivalent( double a, double b, double fps )
Expand Down

0 comments on commit 7a7fed0

Please sign in to comment.