Experiment-driven configuration of PLR/RPLR-based FecController

webrtc/modules/audio_coding/audio_network_adaptor/config.proto

Index: webrtc/modules/audio_coding/audio_network_adaptor/config.proto
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/config.proto b/webrtc/modules/audio_coding/audio_network_adaptor/config.proto
index 6162a155ca85e1eb27777f3172a513db40c3d52b..a20b8536b3f5df45d3fd1191b71dcb6ab6796718 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/config.proto
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/config.proto
@@ -39,6 +39,45 @@ message FecController {
   optional int32 time_constant_ms = 3;
 }
 
+message FecControllerRplrBased {
+  message Threshold {
+    // Threshold defines a curve in the bandwidth/recoverable-packet-loss
+    // domain.
+    // The curve is characterized by the two conjunction points: A and B.
+    //
+    // recoverable ^
+    // packet      |  |
+    //  loss       | A|
+    //             |   \        A: (low_bandwidth_bps,
+    //             |    \           low_bandwidth_recoverable_packet_loss)
+    //             |     \       B: (high_bandwidth_bps,
+    //             |      \          high_bandwidth_recoverable_packet_loss)
+    //             |      B\________
+    //             |---------------> bandwidth
+    optional int32 low_bandwidth_bps = 1;
+    optional float low_bandwidth_recoverable_packet_loss = 2;
+    optional int32 high_bandwidth_bps = 3;
+    optional float high_bandwidth_recoverable_packet_loss = 4;
+  }
+
+  // |fec_enabling_threshold| defines a curve, above which FEC should be
+  // enabled. |fec_disabling_threshold| defines a curve, under which FEC
+  // should be disabled. See below
+  //
+  // packet-loss ^   |  |
+  //             |   |  |   FEC
+  //             |    \  \   ON
+  //             | FEC \  \_______ fec_enabling_threshold
+  //             | OFF  \_________ fec_disabling_threshold
+  //             |-----------------> bandwidth
+  optional Threshold fec_enabling_threshold = 1;
+  optional Threshold fec_disabling_threshold = 2;
+
+  // |time_constant_ms| is the time constant for an exponential filter, which
+  // is used for smoothing the packet loss fraction.
+  optional int32 time_constant_ms = 3;

[michaelt, 2017/03/27 13:52:31]

Isn't FecControllerRplrBased without PL smoothing ?

[elad.alon_webrtc.org, 2017/03/27 14:27:24]

Right you are. Thanks for catching.

+}
+
 message FrameLengthController {
   // Uplink packet loss fraction below which frame length can increase.
   optional float fl_increasing_packet_loss_fraction = 1;
@@ -101,6 +140,7 @@ message Controller {
     ChannelController channel_controller = 23;
     DtxController dtx_controller = 24;
     BitrateController bitrate_controller = 25;
+    FecControllerRplrBased fec_controller_rplr_based = 26;
   }
 }