commit 1f0a4746aa8bfcce3fe7fbd2092aef8fe247aef4
author Deepa K G <deepa.kg@ittiam.com> Tue Oct 29 11:23:09 2019 +0530
committer Sarah Parker <sarahparker@google.com> Wed Oct 30 01:35:08 2019 +0000
tree 87d30d7db2205b47c676166c54d651f04842a5ae
parent 5f74dc225014a7d6184e6192f30764ce1dcf1ea9

Fix assertion failure for single pass

The flag "source_alt_ref_active" was reset
inappropriately for single pass in constant
quality mode (AOM_Q). This patch fixes the
issue.

Change-Id: If01567b88ae9a105b9b67ede5459e868c4b180ef

av1/encoder/ratectrl.c

diff --git a/av1/encoder/ratectrl.c b/av1/encoder/ratectrl.c
index 89d70ba..cd56c1f 100644
--- a/av1/encoder/ratectrl.c
+++ b/av1/encoder/ratectrl.c
@@ -1454,12 +1454,8 @@
     // If we are not using alt ref in the up and coming group clear the arf
     // active flag. In multi arf group case, if the index is not 0 then
     // we are overlaying a mid group arf so should not reset the flag.
-    if (cpi->oxcf.pass == 2) {
-      if (!rc->source_alt_ref_pending && (gf_group->index == 0))
-        rc->source_alt_ref_active = 0;
-    } else if (!rc->source_alt_ref_pending) {
+    if (!rc->source_alt_ref_pending && (gf_group->index == 0))
       rc->source_alt_ref_active = 0;
-    }
   } else if (cpi->common.show_frame) {
     rc->frames_since_golden++;
   }