From ecdedac3dabbbc0345e5a0f98bc2063cd38d236f Mon Sep 17 00:00:00 2001 From: Jakob Ivarsson Date: Tue, 25 Apr 2023 20:42:38 +0200 Subject: [PATCH] Remove NetEq simulation step size restriction. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This should not be relevant anymore and is causing some issues due to SetMinimumDelay events early in the log. Bug: None Change-Id: Ib7e3c624608c9bceaed31bd6669db59887d24659 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/303580 Reviewed-by: Henrik Lundin Commit-Queue: Jakob Ivarsson‎ Cr-Commit-Position: refs/heads/main@{#39956} --- modules/audio_coding/neteq/tools/neteq_test.cc | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/modules/audio_coding/neteq/tools/neteq_test.cc b/modules/audio_coding/neteq/tools/neteq_test.cc index 82dd0ae1c1..ab9a3e2d9a 100644 --- a/modules/audio_coding/neteq/tools/neteq_test.cc +++ b/modules/audio_coding/neteq/tools/neteq_test.cc @@ -273,17 +273,8 @@ NetEqTest::SimulationStepResult NetEqTest::RunToNextGetAudio() { prev_lifetime_stats_ = lifetime_stats; const bool no_more_packets_to_decode = !input_->NextPacketTime() && !operations_state.next_packet_available; - // End the simulation if the gap is too large. This indicates an issue - // with the event log file. - const bool simulation_step_too_large = result.simulation_step_ms > 1000; - if (simulation_step_too_large) { - // If we don't reset the step time, the large gap will be included in - // the simulation time, which can be a large distortion. - result.simulation_step_ms = 10; - } - result.is_simulation_finished = simulation_step_too_large || - no_more_packets_to_decode || - input_->ended(); + result.is_simulation_finished = + no_more_packets_to_decode || input_->ended(); prev_ops_state_ = operations_state; return result; }