summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarco Paniconi <marpan@google.com>2015-11-30 19:00:39 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2015-11-30 19:00:39 +0000
commit23831545a0a148a066b905c0f1e011375c97d790 (patch)
tree33a6eff7b1173416f1fc564da92f9e02704cee95
parentf1f74a4e6c62c10cf5a810d870797184e0661341 (diff)
parentad7e76531950869be24ed3679b9c802bafa85d04 (diff)
downloadlibvpx-23831545a0a148a066b905c0f1e011375c97d790.tar
libvpx-23831545a0a148a066b905c0f1e011375c97d790.tar.gz
libvpx-23831545a0a148a066b905c0f1e011375c97d790.tar.bz2
libvpx-23831545a0a148a066b905c0f1e011375c97d790.zip
Merge "vp9 denoiser: Fix to re-evaluate mode selection."
-rw-r--r--vp9/encoder/vp9_pickmode.c4
-rw-r--r--vp9/encoder/vp9_speed_features.c10
2 files changed, 2 insertions, 12 deletions
diff --git a/vp9/encoder/vp9_pickmode.c b/vp9/encoder/vp9_pickmode.c
index 02e967401..095847a23 100644
--- a/vp9/encoder/vp9_pickmode.c
+++ b/vp9/encoder/vp9_pickmode.c
@@ -1703,8 +1703,7 @@ void vp9_pick_inter_mode(VP9_COMP *cpi, MACROBLOCK *x,
decision == FILTER_BLOCK &&
cpi->noise_estimate.enabled &&
cpi->noise_estimate.level > kLow &&
- zero_last_cost_orig < (best_rdc.rdcost << 2) &&
- !reuse_inter_pred) {
+ zero_last_cost_orig < (best_rdc.rdcost << 2)) {
// Check if we should pick ZEROMV on denoised signal.
int rate = 0;
int64_t dist = 0;
@@ -1713,6 +1712,7 @@ void vp9_pick_inter_mode(VP9_COMP *cpi, MACROBLOCK *x,
mbmi->ref_frame[1] = NONE;
mbmi->mv[0].as_int = 0;
mbmi->interp_filter = EIGHTTAP;
+ xd->plane[0].pre[0] = yv12_mb[LAST_FRAME][0];
vp9_build_inter_predictors_sby(xd, mi_row, mi_col, bsize);
model_rd_for_sb_y(cpi, bsize, x, xd, &rate, &dist, &var_y, &sse_y);
this_rdc.rate = rate + ref_frame_cost[LAST_FRAME] +
diff --git a/vp9/encoder/vp9_speed_features.c b/vp9/encoder/vp9_speed_features.c
index 20516a0c9..318d8100c 100644
--- a/vp9/encoder/vp9_speed_features.c
+++ b/vp9/encoder/vp9_speed_features.c
@@ -380,16 +380,6 @@ static void set_rt_speed_feature(VP9_COMP *cpi, SPEED_FEATURES *sf,
sf->adaptive_rd_thresh = 2;
// This feature is only enabled when partition search is disabled.
sf->reuse_inter_pred_sby = 1;
- // TODO(marpan): When denoising, we may re-evaluate the mode selection and
- // this seems to cause problems when reuse_inter_pred_sby is enabled.
- // Disabling reuse_inter_pred_sby for now (under denoising conditions), and
- // will look into re-enabling it.
-#if CONFIG_VP9_TEMPORAL_DENOISING
- if (cpi->oxcf.noise_sensitivity > 0 &&
- cpi->noise_estimate.enabled &&
- cpi->noise_estimate.level > kLow)
- sf->reuse_inter_pred_sby = 0;
-#endif
sf->partition_search_breakout_rate_thr = 200;
sf->coeff_prob_appx_step = 4;
sf->use_fast_coef_updates = is_keyframe ? TWO_LOOP : ONE_LOOP_REDUCED;