]> granicus.if.org Git - libvpx/commitdiff
Switch the default 2 pass vbr variant.
authorPaul Wilkins <paulwilkins@google.com>
Thu, 1 May 2014 11:10:25 +0000 (12:10 +0100)
committerDeb Mukherjee <debargha@google.com>
Thu, 1 May 2014 18:53:40 +0000 (11:53 -0700)
On balance Deb's  modified rate control for VBR seems
to be outperforming especially on some low motion YT
clips so I have switched this to be the default mode for
now.

Change-Id: I0713d430cad6425ac5c48fccdf332e12814ee44a

vp9/encoder/vp9_firstpass.c

index 1f995c9e7e0f206dab908372f74d4680b5e4e243..5867fb3fc73d3524091ddcd44a7db7113c0651f2 100644 (file)
@@ -61,7 +61,7 @@
 #define MIN_GF_INTERVAL             4
 #endif
 
-#define LONG_TERM_VBR_CORRECTION
+// #define LONG_TERM_VBR_CORRECTION
 
 static void swap_yv12(YV12_BUFFER_CONFIG *a, YV12_BUFFER_CONFIG *b) {
   YV12_BUFFER_CONFIG temp = *a;
@@ -2402,8 +2402,8 @@ void vp9_twopass_postencode_update(VP9_COMP *cpi) {
   const double progress =
       (double)(cpi->twopass.stats_in - cpi->twopass.stats_in_start) /
               (cpi->twopass.stats_in_end - cpi->twopass.stats_in_start);
-  const int bits_used = progress * cpi->rc.this_frame_target +
-                        (1.0 - progress) * cpi->rc.projected_frame_size;
+  const int bits_used = progress * rc->this_frame_target +
+                        (1.0 - progress) * rc->projected_frame_size;
 #endif
 
   cpi->twopass.bits_left -= bits_used;