From 7a9b96ff8e9f8d1f5776300e36b7d79797fac7e2 Mon Sep 17 00:00:00 2001 From: Jerome Jiang Date: Mon, 20 Jul 2020 14:31:34 -0700 Subject: [PATCH] AV1: set error_resilience to 0. No need to keep error_resilience 1 for layers in AV1 Bug: None Change-Id: I6570d653a34ed2187307154ccdfd9e941ed8f917 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/179742 Reviewed-by: Marco Paniconi Commit-Queue: Jerome Jiang Cr-Commit-Position: refs/heads/master@{#31769} --- modules/video_coding/codecs/av1/libaom_av1_encoder.cc | 4 +--- modules/video_coding/codecs/av1/libaom_av1_unittest.cc | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/modules/video_coding/codecs/av1/libaom_av1_encoder.cc b/modules/video_coding/codecs/av1/libaom_av1_encoder.cc index 2371102b95..cd9b37442a 100644 --- a/modules/video_coding/codecs/av1/libaom_av1_encoder.cc +++ b/modules/video_coding/codecs/av1/libaom_av1_encoder.cc @@ -190,9 +190,7 @@ int LibaomAv1Encoder::InitEncode(const VideoCodec* codec_settings, cfg_.rc_min_quantizer = kQpMin; cfg_.rc_max_quantizer = encoder_settings_.qpMax; cfg_.g_usage = kUsageProfile; - if (SvcEnabled()) { - cfg_.g_error_resilient = 1; - } + cfg_.g_error_resilient = 0; // Low-latency settings. cfg_.rc_end_usage = AOM_CBR; // Constant Bit Rate (CBR) mode cfg_.g_pass = AOM_RC_ONE_PASS; // One-pass rate control diff --git a/modules/video_coding/codecs/av1/libaom_av1_unittest.cc b/modules/video_coding/codecs/av1/libaom_av1_unittest.cc index 970d43c066..7a577bc29c 100644 --- a/modules/video_coding/codecs/av1/libaom_av1_unittest.cc +++ b/modules/video_coding/codecs/av1/libaom_av1_unittest.cc @@ -322,7 +322,7 @@ INSTANTIATE_TEST_SUITE_P( /*configured_bitrates=*/ {{{0, 0}, DataRate::KilobitsPerSec(70)}, {{0, 1}, DataRate::KilobitsPerSec(30)}, - {{1, 0}, DataRate::KilobitsPerSec(140)}, + {{1, 0}, DataRate::KilobitsPerSec(110)}, {{1, 1}, DataRate::KilobitsPerSec(80)}}}), [](const testing::TestParamInfo& info) { return info.param.name;