Fix bug in use of zoom metric as part of arf breakout.
authorpaulwilkins <paulwilkins@google.com>
Thu, 4 Jan 2018 16:02:42 +0000 (16:02 +0000)
committerJames Zern <jzern@google.com>
Tue, 16 Jan 2018 19:21:29 +0000 (11:21 -0800)
commit7d197399492ab417ab8ba76fff9aa1b5309edd36
tree03a459f9ec196058f2aa25982afac7f25be46be5
parentc5dc3373dbd442ea299bacf276c4258fa7ce0559
Fix bug in use of zoom metric as part of arf breakout.

The in/out (or zoom metrics) in accumulate_frame_motion_stats()
are in effect a % of the blocks that have a motion vector pointing
either towards or away from the center. As such they are already
normalized in terms of image size and the thresholds against which
these are tested should be image size independent.

In practice a zoom either in or out is an indicator for a shorter group
length so the abs value is more important as a breakout clause.

This patch fixes the threshold test. Clips without noticeable zoom show
no effect but some  with strong zooms such as "station" show a big
gain (5-10%). Average psnr-hvs gain on hdres set was 0.292%

Change-Id: I4f97a72b0e273e4e844ade15285749c32cd81c1c
(cherry picked from commit 0226ce79e9389ccf7d10ed7acacba6840ad911c9)
vp9/encoder/vp9_firstpass.c