diff options
author | Paul Wilkins <paulwilkins@google.com> | 2014-02-13 14:50:55 +0000 |
---|---|---|
committer | Paul Wilkins <paulwilkins@google.com> | 2014-02-13 14:51:38 +0000 |
commit | c30a9cd97dbeee9cdd18527e9571304cb01d0073 (patch) | |
tree | 25a1ad3fbf6b68ac045aeb7b5fb9ef2a7e5560a6 /vp9/encoder | |
parent | f30b323180c3c358b646414f79f33da5664eeccb (diff) | |
download | libvpx-c30a9cd97dbeee9cdd18527e9571304cb01d0073.tar libvpx-c30a9cd97dbeee9cdd18527e9571304cb01d0073.tar.gz libvpx-c30a9cd97dbeee9cdd18527e9571304cb01d0073.tar.bz2 libvpx-c30a9cd97dbeee9cdd18527e9571304cb01d0073.zip |
Remove some cases for turning off arf.
Flipping arf on and off too often is hurting some clips.
This change makes no difference for 50-75% of our test
clips but helps some by a big margin. (eg. std-hd crew
by 6% and one of the YT and YT-hd clips by 14%)
Average improvements for 2 pass, speed 2 (psnr,ssim)
are as follows:-
derf 0.165%, 0.210%
yt 1.210%, 1.464%
yt-hd 1.189%, 1.471%
std-hd 1.031%, 0.886%
Change-Id: I121fe66cfb4a62d384b23b484a7d648789641969
Diffstat (limited to 'vp9/encoder')
-rw-r--r-- | vp9/encoder/vp9_firstpass.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/vp9/encoder/vp9_firstpass.c b/vp9/encoder/vp9_firstpass.c index 838f74e8c..509dee838 100644 --- a/vp9/encoder/vp9_firstpass.c +++ b/vp9/encoder/vp9_firstpass.c @@ -1566,13 +1566,7 @@ static void define_gf_group(VP9_COMP *cpi, FIRSTPASS_STATS *this_frame) { (i >= MIN_GF_INTERVAL) && // for real scene cuts (not forced kfs) dont allow arf very near kf. (rc->next_key_frame_forced || - (i <= (rc->frames_to_key - MIN_GF_INTERVAL))) && - ((next_frame.pcnt_inter > 0.75) || - (next_frame.pcnt_second_ref > 0.5)) && - ((mv_in_out_accumulator / (double)i > -0.2) || - (mv_in_out_accumulator > -2.0)) && - (boost_score > 100)) { - + (i <= (rc->frames_to_key - MIN_GF_INTERVAL)))) { // Alternative boost calculation for alt ref rc->gfu_boost = calc_arf_boost(cpi, 0, (i - 1), (i - 1), &f_boost, &b_boost); |