From b095d9df3c7492e9ba031c4491a0a565f668c9e5 Mon Sep 17 00:00:00 2001 From: Paul Wilkins Date: Tue, 4 Jan 2011 17:55:49 +0000 Subject: [PATCH] Adjustment to boost calculation in two pass. Calculate a minimum intra value to be used in determining the IIratio scores used in two pass, second pass. This is to make sure sections that are low complexity" in the intra domain are still boosted appropriately for KF/GF/ARF. For now I have commented out the Q based adjustment of KF boost. Change-Id: I15deb09c5bd9b53180a2ddd3e5f575b2aba244b3 --- vp8/encoder/firstpass.c | 29 +++++++++++++++++++++++++---- vp8/encoder/onyx_int.h | 2 ++ 2 files changed, 27 insertions(+), 4 deletions(-) diff --git a/vp8/encoder/firstpass.c b/vp8/encoder/firstpass.c index 5cb1fd9b1..da4d740cb 100644 --- a/vp8/encoder/firstpass.c +++ b/vp8/encoder/firstpass.c @@ -53,9 +53,11 @@ extern const int vp8_gf_boost_qadjustment[QINDEX_RANGE]; #define IIFACTOR 1.4 #define IIKFACTOR1 1.40 #define IIKFACTOR2 1.5 -#define RMAX 14.0 -#define GF_RMAX 48.0 // 128.0 +#define RMAX 14.0 +#define GF_RMAX 48.0 +#define KF_MB_INTRA_MIN 300 +#define GF_MB_INTRA_MIN 200 #define DOUBLE_DIVIDE_CHECK(X) ((X)<0?(X)-.000001:(X)+.000001) #define POW1 (double)cpi->oxcf.two_pass_vbrbias/100.0 @@ -1147,6 +1149,13 @@ void vp8_init_second_pass(VP8_COMP *cpi) cpi->bits_left -= (long long)(cpi->total_stats->duration * two_pass_min_rate / 10000000.0); cpi->clip_bits_total = cpi->bits_left; + // Calculate a minimum intra value to be used in determining the IIratio + // scores used in the second pass. We have this minimum to make sure + // that clips that are static but "low complexity" in the intra domain + // are still boosted appropriately for KF/GF/ARF + cpi->kf_intra_err_min = KF_MB_INTRA_MIN * cpi->common.MBs; + cpi->gf_intra_err_min = GF_MB_INTRA_MIN * cpi->common.MBs; + vp8_avg_stats(cpi->total_stats); // Scan the first pass file and calculate an average Intra / Inter error score ratio for the sequence @@ -1317,6 +1326,13 @@ static void define_gf_group(VP8_COMP *cpi, FIRSTPASS_STATS *this_frame) // Underlying boost factor is based on inter intra error ratio r = (boost_factor * (next_frame.intra_error / DOUBLE_DIVIDE_CHECK(next_frame.coded_error))); + if (next_frame.intra_error > cpi->gf_intra_err_min) + r = (IIKFACTOR2 * next_frame.intra_error / + DOUBLE_DIVIDE_CHECK(next_frame.coded_error)); + else + r = (IIKFACTOR2 * cpi->gf_intra_err_min / + DOUBLE_DIVIDE_CHECK(next_frame.coded_error)); + // Increase boost for frames where new data coming into frame (eg zoom out) // Slightly reduce boost if there is a net balance of motion out of the frame (zoom in) // The range for this_frame_mv_in_out is -1.0 to +1.0 @@ -2302,7 +2318,12 @@ void vp8_find_next_key_frame(VP8_COMP *cpi, FIRSTPASS_STATS *this_frame) if (EOF == vp8_input_stats(cpi, &next_frame)) break; - r = (IIKFACTOR2 * next_frame.intra_error / DOUBLE_DIVIDE_CHECK(next_frame.coded_error)) ; + if (next_frame.intra_error > cpi->kf_intra_err_min) + r = (IIKFACTOR2 * next_frame.intra_error / + DOUBLE_DIVIDE_CHECK(next_frame.coded_error)); + else + r = (IIKFACTOR2 * cpi->kf_intra_err_min / + DOUBLE_DIVIDE_CHECK(next_frame.coded_error)); if (r > RMAX) r = RMAX; @@ -2443,7 +2464,7 @@ void vp8_find_next_key_frame(VP8_COMP *cpi, FIRSTPASS_STATS *this_frame) kf_boost = (int)((double)kf_boost * 100.0) >> 4; // Scale 16 to 100 // Adjustment to boost based on recent average q - kf_boost = kf_boost * vp8_kf_boost_qadjustment[cpi->ni_av_qi] / 100; + //kf_boost = kf_boost * vp8_kf_boost_qadjustment[cpi->ni_av_qi] / 100; if (kf_boost < 250) // Min KF boost kf_boost = 250; diff --git a/vp8/encoder/onyx_int.h b/vp8/encoder/onyx_int.h index ab270ca5f..2f9cc4776 100644 --- a/vp8/encoder/onyx_int.h +++ b/vp8/encoder/onyx_int.h @@ -479,6 +479,8 @@ typedef struct double total_coded_error_left; double start_tot_err_left; double min_error; + double kf_intra_err_min; + double gf_intra_err_min; double modified_error_total; double modified_error_used;