]> granicus.if.org Git - libvpx/commitdiff
vp9 screen-content: Keep lower step_param for quality layers.
authorMarco Paniconi <marpan@google.com>
Wed, 21 Nov 2018 23:01:04 +0000 (15:01 -0800)
committerMarco Paniconi <marpan@google.com>
Thu, 22 Nov 2018 00:21:58 +0000 (16:21 -0800)
Issue with step_param = 2 seems specific to lower layers
with different resolution.

Change-Id: I26405488ac7691b3e471e98e794d4b1d8098a91d

vp9/encoder/vp9_speed_features.c

index 9ea84f365484a48867f5426c54b9d82030285292..fbd24d0d2b0db9c848678257f4b0bdae6ce2066a 100644 (file)
@@ -804,8 +804,12 @@ static void set_rt_speed_feature_framesize_independent(
       cpi->svc.spatial_layer_id == 0 &&
       (cpi->rc.high_num_blocks_with_motion || cpi->svc.last_layer_dropped[0])) {
     sf->mv.search_method = NSTEP;
-    // TODO(marpan/jianj): Investigate issue for lower setting of step_param.
-    sf->mv.fullpel_search_step_param = 4;
+    sf->mv.fullpel_search_step_param = 2;
+    // TODO(marpan/jianj): Investigate issue for lower setting of step_param
+    // for spatial layers (namely on lower layers).
+    if (cpi->use_svc && cm->width != cpi->oxcf.width &&
+        cm->height != cpi->oxcf.height)
+      sf->mv.fullpel_search_step_param = 4;
   }
 }