commit 2ef786810ceb19ab561301d4fe85a8b0afc8eadb
author chiyotsai <chiyotsai@google.com> Tue Oct 01 10:39:28 2019 -0700
committer Chi Yo Tsai <chiyotsai@google.com> Wed Oct 02 23:08:53 2019 +0000
tree 08ae63ba6367dee0a85f5af149029c7a3fca4178
parent 7de6a48d7753a32c1c06148f55f7b7f8b73cc7ac

Conditionally use simple_motion_search for screen content

This commit disables PARTITION_SPLIT pruning with simple_motion_search
when the current video is screen content on spd 1 and 2. This makes the
behavior consistent with the behavior on spd 3+.

STATS_CHANGED

Change-Id: I655bf1df9d3726c5a3ac14333d5667919f2a8926

av1/encoder/speed_features.c

diff --git a/av1/encoder/speed_features.c b/av1/encoder/speed_features.c
index d0c8bf9..54e108d 100644
--- a/av1/encoder/speed_features.c
+++ b/av1/encoder/speed_features.c
@@ -148,15 +148,12 @@
   }
 
   if (speed >= 1) {
-    sf->simple_motion_search_split = 2;
     if (is_720p_or_larger) {
       sf->use_square_partition_only_threshold = BLOCK_128X128;
     } else if (is_480p_or_larger) {
       sf->use_square_partition_only_threshold = BLOCK_64X64;
     } else {
       sf->use_square_partition_only_threshold = BLOCK_32X32;
-
-      sf->simple_motion_search_split = 1;
     }
 
     if (!is_720p_or_larger) {
@@ -199,14 +196,6 @@
       sf->partition_search_breakout_dist_thr = (1 << 23);
       sf->partition_search_breakout_rate_thr = 120;
     }
-
-    // TODO(Venkat): Clean-up frame type dependency for
-    // simple_motion_search_split in partition search function and set the
-    // speed feature accordingly
-    // TODO(any): The models and thresholds used by simple_motion_split is
-    // trained and tuned on speed 1 and 2. We might get better performance if we
-    // readjust them for speed 3 and 4.
-    sf->simple_motion_search_split = cm->allow_screen_content_tools ? 1 : 2;
   }
 
   if (speed >= 4) {
@@ -299,7 +288,10 @@
     sf->prune_comp_search_by_single_result = 1;
     sf->skip_repeated_newmv = 1;
     sf->obmc_full_pixel_search_level = 1;
-    sf->simple_motion_search_split = 1;
+    // TODO(Venkat): Clean-up frame type dependency for
+    // simple_motion_search_split in partition search function and set the
+    // speed feature accordingly
+    sf->simple_motion_search_split = cm->allow_screen_content_tools ? 1 : 2;
     sf->simple_motion_search_early_term_none = 1;
 
     sf->disable_wedge_search_var_thresh = 0;