]> granicus.if.org Git - libvpx/commitdiff
Revert "vp8: Increase rate threshold for overshoot-drop"
authorMarco Paniconi <marpan@google.com>
Thu, 11 Oct 2018 10:24:16 +0000 (10:24 +0000)
committerMarco Paniconi <marpan@google.com>
Thu, 11 Oct 2018 10:24:16 +0000 (10:24 +0000)
This reverts commit bc066684ca4deff24d02ee56071d731b431bf438.

Reason for revert: <INSERT REASONING HERE>
Regression in webrtc perf test

Original change's description:
> vp8: Increase rate threshold for overshoot-drop
>
> Increase the rate threshold for the dropping when
> overshoot is detected during encoding. This helps
> to prevent some unneccessary drops for hard content.
>
> Change-Id: I258bf33883d46347efd44e1e192cb25c444d05fe

TBR=sprang@chromium.org,marpan@google.com,builds@webmproject.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Change-Id: Ib0e84747430ba6d04e479f9efd86d628b80a1e67

vp8/encoder/ratectrl.c

index e89f71ed8fb4e49e5778bb951f165d5ed9561378..fc833bccc9445ef08eb12caeb7f6fc8669e0860a 100644 (file)
@@ -1474,7 +1474,7 @@ int vp8_drop_encodedframe_overshoot(VP8_COMP *cpi, int Q) {
     // QP threshold: only allow dropping if we are not close to qp_max.
     int thresh_qp = 3 * cpi->worst_quality >> 2;
     // Rate threshold, in bytes.
-    int thresh_rate = 3 * (cpi->av_per_frame_bandwidth >> 3);
+    int thresh_rate = 2 * (cpi->av_per_frame_bandwidth >> 3);
     // Threshold for the average (over all macroblocks) of the pixel-sum
     // residual error over 16x16 block.
     int thresh_pred_err_mb = (200 << 4);