From 3472cbbd8862bb5e0256ef2d8a773170ccda8bda Mon Sep 17 00:00:00 2001 From: Marco Date: Thu, 4 Feb 2016 17:10:25 -0800 Subject: [PATCH] vp9 aq-mode=3: Keep it on even at low bitrates. Change-Id: I6a27db6b9dbe3d8cb3d47ef30fbd52557171cadd --- vp9/encoder/vp9_aq_cyclicrefresh.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/vp9/encoder/vp9_aq_cyclicrefresh.c b/vp9/encoder/vp9_aq_cyclicrefresh.c index 0982fd6ff..4c75ffaa9 100644 --- a/vp9/encoder/vp9_aq_cyclicrefresh.c +++ b/vp9/encoder/vp9_aq_cyclicrefresh.c @@ -60,8 +60,6 @@ void vp9_cyclic_refresh_free(CYCLIC_REFRESH *cr) { } // Check if we should turn off cyclic refresh based on bitrate condition. -// TODO(marpan): May be better in some cases to just reduce the amount/delta-qp -// instead of completely shutting off. static int apply_cyclic_refresh_bitrate(const VP9_COMMON *cm, const RATE_CONTROL *rc) { // Turn off cyclic refresh if bits available per frame is not sufficiently @@ -535,7 +533,10 @@ void vp9_cyclic_refresh_setup(VP9_COMP *const cpi) { const RATE_CONTROL *const rc = &cpi->rc; CYCLIC_REFRESH *const cr = cpi->cyclic_refresh; struct segmentation *const seg = &cm->seg; - const int apply_cyclic_refresh = apply_cyclic_refresh_bitrate(cm, rc); + // TODO(marpan): Look into whether we should reduce the amount/delta-qp + // instead of completely shutting off at low bitrates. For now keep it on. + // const int apply_cyclic_refresh = apply_cyclic_refresh_bitrate(cm, rc); + const int apply_cyclic_refresh = 1; if (cm->current_video_frame == 0) cr->low_content_avg = 0.0; // Don't apply refresh on key frame or temporal enhancement layer frames.