commit 7e915ab90bf798e736048d695d7041515e0b9e60
author Yeqing Wu <yeqing_wu@apple.com> Fri Jul 09 20:38:43 2021 -0700
committer Yunqing Wang <yunqingwang@google.com> Tue Jul 13 17:32:26 2021 +0000
tree 5f01eb6a3a6e57c31ba395a2a35a97de8e83e917
parent 053c49cc0d31854fc3150380dbe921aeafad602b

Bugfix motion field estimation

Based on the logic on encoder side and the syntax on the AV1 spec,
the motion field estimation should be turned on or off based on
the control flag allow_ref_frame_mvs instead of always enable it
on the decoder. If encoder's running time flag --enable-ref-frame-mvs
is set to 0 (disable), then the motion field estimation on encoder is
disabled, however the motion field estimation on decoder is still
enabled due to this bug, which will cause the inconsistent between
encoder and decoder.

Change-Id: I339cbb747b967dafa73e6f2b5cd192acfbe595aa

av1/decoder/decodeframe.c

diff --git a/av1/decoder/decodeframe.c b/av1/decoder/decodeframe.c
index 0ea2873..ef14bf6 100644
--- a/av1/decoder/decodeframe.c
+++ b/av1/decoder/decodeframe.c
@@ -5155,7 +5155,7 @@
 
   cm->mi_params.setup_mi(&cm->mi_params);
 
-  av1_setup_motion_field(cm);
+  if (cm->features.allow_ref_frame_mvs) av1_setup_motion_field(cm);
 
   av1_setup_block_planes(xd, cm->seq_params->subsampling_x,
                          cm->seq_params->subsampling_y, num_planes);