diff options
author | Marco Paniconi <marpan@google.com> | 2015-11-30 19:00:39 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2015-11-30 19:00:39 +0000 |
commit | 23831545a0a148a066b905c0f1e011375c97d790 (patch) | |
tree | 33a6eff7b1173416f1fc564da92f9e02704cee95 | |
parent | f1f74a4e6c62c10cf5a810d870797184e0661341 (diff) | |
parent | ad7e76531950869be24ed3679b9c802bafa85d04 (diff) | |
download | libvpx-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.c | 4 | ||||
-rw-r--r-- | vp9/encoder/vp9_speed_features.c | 10 |
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; |