tree d85b2521a5e6e6cfa31d8391a54151895eaef32d
parent ab578287e69bc9325f6dee01165ab716eada11ff
author Ranjit Kumar Tulabandu <ranjit.tulabandu@ittiam.com> 1671103999 +0530
committer Yunqing Wang <yunqingwang@google.com> 1671423268 +0000

Fix issues with population for the sf recode_tolerance

The speed feature recode_tolerance was set to 55 in
set_good_speed_features_framesize_independent() for
speed=6 in the following CL:
https://aomedia-review.googlesource.com/c/aom/+/123883

Later the speed feature was set based on resolution in
set_good_speed_feature_framesize_dependent() for
speed<=5 in the following CL:
https://aomedia-review.googlesource.com/c/aom/+/132361

As set_good_speed_feature_framesize_dependent() is called
after set_good_speed_features_framesize_independent(), the
speed feature value was being overwritten with unintended
values for speed=6, >= 720p resolutions. In this CL, the
speed feature setting is moved to
set_good_speed_feature_framesize_dependent() to avoid above
issue.

For 'good' encoding,
               Instruction Count        BD-Rate Loss(%)
cpu Resolution   Reduction(%)    avg.psnr  ovr.psnr   ssim
 6    HDRES2       0.481         0.0708     0.0812   0.0083

STATS_CHANGED for 'good', speed=6, >= 720p resolutions

Change-Id: I10a30535aaad1bddb43faceb7876162d08f5fa02