Commit 1df7ef29 authored by Deb Mukherjee's avatar Deb Mukherjee

Remove one shot q experiment

The experiment is no longer used and can be removed.

Change-Id: I9feab378fc895c120aa375353c68f93cad090609
parent b26ce8b1
...@@ -249,7 +249,6 @@ HAVE_LIST=" ...@@ -249,7 +249,6 @@ HAVE_LIST="
unistd_h unistd_h
" "
EXPERIMENT_LIST=" EXPERIMENT_LIST="
oneshotq
multiple_arf multiple_arf
non420 non420
alpha alpha
......
...@@ -2169,17 +2169,14 @@ void vp9_second_pass(VP9_COMP *cpi) { ...@@ -2169,17 +2169,14 @@ void vp9_second_pass(VP9_COMP *cpi) {
cpi->ni_av_qi = tmp_q; cpi->ni_av_qi = tmp_q;
cpi->avg_q = vp9_convert_qindex_to_q(tmp_q); cpi->avg_q = vp9_convert_qindex_to_q(tmp_q);
#ifndef ONE_SHOT_Q_ESTIMATE
// Limit the maxq value returned subsequently. // Limit the maxq value returned subsequently.
// This increases the risk of overspend or underspend if the initial // This increases the risk of overspend or underspend if the initial
// estimate for the clip is bad, but helps prevent excessive // estimate for the clip is bad, but helps prevent excessive
// variation in Q, especially near the end of a clip // variation in Q, especially near the end of a clip
// where for example a small overspend may cause Q to crash // where for example a small overspend may cause Q to crash
adjust_maxq_qrange(cpi); adjust_maxq_qrange(cpi);
#endif
} }
#ifndef ONE_SHOT_Q_ESTIMATE
// The last few frames of a clip almost always have to few or too many // The last few frames of a clip almost always have to few or too many
// bits and for the sake of over exact rate control we dont want to make // bits and for the sake of over exact rate control we dont want to make
// radical adjustments to the allowed quantizer range just to use up a // radical adjustments to the allowed quantizer range just to use up a
...@@ -2202,7 +2199,6 @@ void vp9_second_pass(VP9_COMP *cpi) { ...@@ -2202,7 +2199,6 @@ void vp9_second_pass(VP9_COMP *cpi) {
cpi->active_worst_quality = cpi->active_worst_quality =
adjust_active_maxq(cpi->active_worst_quality, tmp_q); adjust_active_maxq(cpi->active_worst_quality, tmp_q);
} }
#endif
} }
vp9_zero(this_frame); vp9_zero(this_frame);
if (EOF == input_stats(cpi, &this_frame)) if (EOF == input_stats(cpi, &this_frame))
......
...@@ -2856,19 +2856,11 @@ static int pick_q_and_adjust_q_bounds(VP9_COMP *cpi, ...@@ -2856,19 +2856,11 @@ static int pick_q_and_adjust_q_bounds(VP9_COMP *cpi,
if (cpi->oxcf.end_usage == USAGE_CONSTANT_QUALITY) { if (cpi->oxcf.end_usage == USAGE_CONSTANT_QUALITY) {
cpi->active_best_quality = cpi->cq_target_quality; cpi->active_best_quality = cpi->cq_target_quality;
} else { } else {
#ifdef ONE_SHOT_Q_ESTIMATE
#ifdef STRICT_ONE_SHOT_Q
cpi->active_best_quality = q;
#else
cpi->active_best_quality = inter_minq[q];
#endif
#else
cpi->active_best_quality = inter_minq[q]; cpi->active_best_quality = inter_minq[q];
// 1-pass: for now, use the average Q for the active_best, if its lower // 1-pass: for now, use the average Q for the active_best, if its lower
// than active_worst. // than active_worst.
if (cpi->pass == 0 && (cpi->avg_frame_qindex < q)) if (cpi->pass == 0 && (cpi->avg_frame_qindex < q))
cpi->active_best_quality = inter_minq[cpi->avg_frame_qindex]; cpi->active_best_quality = inter_minq[cpi->avg_frame_qindex];
#endif
// For the constrained quality mode we don't want // For the constrained quality mode we don't want
// q to fall below the cq level. // q to fall below the cq level.
......
...@@ -29,11 +29,6 @@ ...@@ -29,11 +29,6 @@
#include "vp9/common/vp9_findnearmv.h" #include "vp9/common/vp9_findnearmv.h"
#include "vp9/encoder/vp9_lookahead.h" #include "vp9/encoder/vp9_lookahead.h"
// Experimental rate control switches
#if CONFIG_ONESHOTQ
#define ONE_SHOT_Q_ESTIMATE 0
#define STRICT_ONE_SHOT_Q 0
#endif
#define DISABLE_RC_LONG_TERM_MEM 0 #define DISABLE_RC_LONG_TERM_MEM 0
// #define MODE_TEST_HIT_STATS // #define MODE_TEST_HIT_STATS
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment