commit adecfb681f7bbca40f558d458652f8791307f953
author Emil Vardar <vardar@google.com> Fri Jul 11 07:58:39 2025
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Jul 11 11:01:42 2025
tree e5d3815f1e37aef30ebb910a6d7046f33e4bad47
parent 9f42d00a217052b1435def32ba4630aefd5a3e01

Fix discrepancy when std_dev = 0.

Bug: webrtc:431021030
Change-Id: Ib19ce5a1d7955e93ae9eac2d7afe772b5263831f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/400020
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Fanny Linderborg <linderborg@webrtc.org>
Reviewed-by: Fanny Linderborg <linderborg@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#45136}

video/corruption_detection/halton_frame_sampler.cc

diff --git a/video/corruption_detection/halton_frame_sampler.cc b/video/corruption_detection/halton_frame_sampler.cc
index afedb42..4efadcf 100644
--- a/video/corruption_detection/halton_frame_sampler.cc
+++ b/video/corruption_detection/halton_frame_sampler.cc
@@ -111,8 +111,14 @@
   RTC_CHECK_LT(row, frame_sampler.height(channel));
   RTC_CHECK_GE(column, 0);
   RTC_CHECK_LT(column, frame_sampler.width(channel));
-  RTC_CHECK_GT(std_dev, 0.0)
-      << "Standard deviation = 0 yields improper Gaussian weights.";
+  RTC_CHECK_GE(std_dev, 0.0);
+
+  // `std_dev` being zero should ideally correspond to a very low QP value. In
+  // this case even a noisy pixel should be able to be encoded and transmitted
+  // correctly. Hence, the pixel value can be used as is.
+  if (std_dev == 0.0) {
+    return frame_sampler.GetSampleValue(channel, column, row);
+  }
 
   int max_distance =
       std::ceil(std::sqrt(-2.0 * std::log(kCutoff) * std::pow(std_dev, 2.0))) -