Commit 32fa66dd23 for aom
commit 32fa66dd23811e4ae4a443a0a455c84e59ebb7f8
Author: Lin Zheng <linzhen@google.com>
Date: Fri Mar 6 18:04:20 2026 +0000
Fix BasicRateTargetingVBRLagRealtime after 2fed9c
The unit test "BasicRateTargetingVBRLagRealtime()" fails after commit
2fed9c. With CONFIG_REALTIME_ONLY=1, lag_in_frames is set to 0, and
along with other factors in that configuration the threshold in that
specific unit test needs to be lowered.
Change-Id: I2170bef666bcb1fcc4a9a3e68f9a617bf71daeb5
diff --git a/test/datarate_test.cc b/test/datarate_test.cc
index c777d4a9c0..d886c2a942 100644
--- a/test/datarate_test.cc
+++ b/test/datarate_test.cc
@@ -373,8 +373,16 @@ class DatarateTestLarge
ResetModel();
lag_realtime_mode_ = 1;
ASSERT_NO_FATAL_FAILURE(RunLoop(&video));
+#if CONFIG_REALTIME_ONLY
+ // With CONFIG_REALTIME_ONLY=1, lag_in_frames is set to 0, and along with
+ // other factors in that configuration the threshold needs to be lowered
+ // after commit 2fed9c389d.
+ ASSERT_GE(effective_datarate_, cfg_.rc_target_bitrate * 0.65)
+ << " The datarate for the file is lower than target by too much!";
+#else
ASSERT_GE(effective_datarate_, cfg_.rc_target_bitrate * 0.85)
<< " The datarate for the file is lower than target by too much!";
+#endif
ASSERT_LE(effective_datarate_, cfg_.rc_target_bitrate * 2.0)
<< " The datarate for the file is greater than target by too much!";
}