shithub: libvpx

Download patch

ref: 9586d5e682683cc977f655e78b9ac937fc51eede
parent: 2c3a2ad6f1a01a150d4bce5f57e7e04090e71e4b
author: Marco <marpan@google.com>
date: Mon May 8 06:47:53 EDT 2017

vp9: SVC: Modify conditon for setting downsample filter type.

Base the condition on the resolution of the spatial layer.
And remove restriction on scaling factor.

Change-Id: Iad00177ce364279d85661654bff00ce7f48a672e

--- a/vp9/encoder/vp9_svc_layercontext.c
+++ b/vp9/encoder/vp9_svc_layercontext.c
@@ -651,15 +651,15 @@
                        lc->scaling_factor_num, lc->scaling_factor_den, &width,
                        &height);
 
-  // For 3 spatial layers, on the lowest resolution layer: set the filtertype
-  // for downsampling source to 1, to get averaging filter.
-  if (cpi->svc.number_spatial_layers == 3)
-    cpi->svc.filtertype_downsample_source[0] = 1;
+  // For low resolutions: set the filtertype for downsampling source to 1,
+  // to get averaging filter.
+  if (width <= 320 && height <= 240)
+    cpi->svc.filtertype_downsample_source[cpi->svc.spatial_layer_id] = 1;
+
   // The usage of use_base_mv assumes down-scale of 2x2. For now, turn off use
   // of base motion vectors if spatial scale factors for any layers are not 2,
   // keep the case of 3 spatial layers with scale factor of 4x4 for base layer.
   // TODO(marpan): Fix this to allow for use_base_mv for scale factors != 2.
-  // Same condition applies to use of non-zero phase_scaler.
   if (cpi->svc.number_spatial_layers > 1) {
     int sl;
     for (sl = 0; sl < cpi->svc.number_spatial_layers - 1; ++sl) {
@@ -668,10 +668,7 @@
       if ((lc->scaling_factor_num != lc->scaling_factor_den >> 1) &&
           !(lc->scaling_factor_num == lc->scaling_factor_den >> 2 && sl == 0 &&
             cpi->svc.number_spatial_layers == 3)) {
-        int sl2;
         cpi->svc.use_base_mv = 0;
-        for (sl2 = 0; sl2 < cpi->svc.number_spatial_layers - 1; ++sl2)
-          cpi->svc.filtertype_downsample_source[sl2] = 0;
         break;
       }
     }