New statistics interface for APM

webrtc/modules/audio_processing/include/audio_processing.h

Index: webrtc/modules/audio_processing/include/audio_processing.h
diff --git a/webrtc/modules/audio_processing/include/audio_processing.h b/webrtc/modules/audio_processing/include/audio_processing.h
index 0fdbfd2d75cf48caa50bb8b2f8a86f5e156ee353..603e1d85f4bf760a208e7f8214f7225f5f56c7c5 100644
--- a/webrtc/modules/audio_processing/include/audio_processing.h
+++ b/webrtc/modules/audio_processing/include/audio_processing.h
@@ -469,6 +469,85 @@ class AudioProcessing {
   // specific member variables are reset.
   virtual void UpdateHistogramsOnCallEnd() = 0;
 
+  // TODO(ivoc): Remove when the calling code no longer uses the old Statistics
+  //             API.
+  struct Statistic {
+    int instant = 0;  // Instantaneous value.
+    int average = 0;  // Long-term average.
+    int maximum = 0;  // Long-term maximum.
+    int minimum = 0;  // Long-term minimum.
+  };
+
+  struct ValueStatistic {

[the sun, 2016/10/24 15:15:43]

How about "Stat"?

I'm thinking in the future we may want to templatize the type (Statzfloat>,
Stat<int>, etc) an then this will feel unnecessarily long. Don't make it a
template right now.

[ivoc, 2016/10/24 15:50:42]

Sounds good.

+    ValueStatistic() = default;
+    ValueStatistic(ValueStatistic& other) {

[the sun, 2016/10/24 15:15:44]

why not "default"?

[ivoc, 2016/10/24 15:50:42]

Turns out it's not actually necessary to define this or the constructor at all.
I got a message about it previously, but with the current changes it seems to no
longer be needed.

[the sun, 2016/10/25 06:44:12]

have you tried building everywhere? clang complains if the default methods
generate too much code - in which case you may need to declare them and put the
definition in the .cc

[ivoc, 2016/10/25 09:38:45]

It seems to build fine on the trybots, so I guess the class is small enough that
this is not an issue.

+      instant_ = other.instant();
+      average_ = other.average();
+      maximum_ = other.maximum();
+      minimum_ = other.minimum();
+    }
+    ValueStatistic(Statistic& other) {

[the sun, 2016/10/24 15:15:43]

Make the conversion explicit

[ivoc, 2016/10/24 15:50:42]

I changed this into another Set function that takes an old Statistics object, is
that okay?

[the sun, 2016/10/25 06:44:12]

sgtm

+      instant_ = other.instant;
+      average_ = other.average;
+      maximum_ = other.maximum;
+      minimum_ = other.minimum;
+    }
+
+    void SetValues(float instant, float average, float maximum, float minimum) {

[the sun, 2016/10/24 15:15:44]

nit: Just "Set"

[ivoc, 2016/10/24 15:50:42]

Done.

+      RTC_DCHECK_LE(instant, maximum);
+      RTC_DCHECK_GE(instant, minimum);
+      RTC_DCHECK_LE(average, maximum);
+      RTC_DCHECK_GE(average, minimum);
+      instant_ = instant;
+      average_ = average;
+      maximum_ = maximum;
+      minimum_ = minimum;
+    }
+    float instant() const { return instant_; }
+    float average() const { return average_; }
+    float maximum() const { return maximum_; }
+    float minimum() const { return minimum_; }
+
+   private:
+    float instant_ = 0.0f;  // Instantaneous value.
+    float average_ = 0.0f;  // Long-term average.
+    float maximum_ = 0.0f;  // Long-term maximum.
+    float minimum_ = 0.0f;  // Long-term minimum.
+  };
+
+  struct AudioProcessingStatistics {
+    // AEC Statistics.
+    // RERL = ERL + ERLE
+    ValueStatistic residual_echo_return_loss;
+    // ERL = 10log_10(P_far / P_echo)
+    ValueStatistic echo_return_loss;
+    // ERLE = 10log_10(P_echo / P_out)
+    ValueStatistic echo_return_loss_enhancement;
+    // (Pre non-linear processing suppression) A_NLP = 10log_10(P_echo / P_a)
+    ValueStatistic a_nlp;
+    // Fraction of time that the AEC linear filter is divergent, in a 1-second
+    // non-overlapped aggregation window.
+    float divergent_filter_fraction = 0.0f;
+
+    // The delay metrics consists of the delay median and standard deviation. It
+    // also consists of the fraction of delay estimates that can make the echo
+    // cancellation perform poorly. The values are aggregated until the first
+    // call to |GetStatistics()| and afterwards aggregated and updated every
+    // second. Note that if there are several clients pulling metrics from
+    // |GetStatistics()| during a session the first call from any of them will
+    // change to one second aggregation window for all.
+    int delay_median = 0;
+    int delay_standard_deviation = 0;
+    float fraction_poor_delays = 0.0f;
+
+    // Residual echo detector likelihood. This value is not yet calculated and
+    // is currently always set to zero.
+    // TODO(ivoc): Implement this stat.
+    float residual_echo_likelihood = 0.0f;
+  };
+
+  virtual AudioProcessingStatistics GetStatistics() const = 0;
+
   // These provide access to the component interfaces and should never return
   // NULL. The pointers will be valid for the lifetime of the APM instance.
   // The memory for these objects is entirely managed internally.
@@ -480,13 +559,6 @@ class AudioProcessing {
   virtual NoiseSuppression* noise_suppression() const = 0;
   virtual VoiceDetection* voice_detection() const = 0;
 
-  struct Statistic {
-    int instant;  // Instantaneous value.
-    int average;  // Long-term average.
-    int maximum;  // Long-term maximum.
-    int minimum;  // Long-term minimum.
-  };
-
   enum Error {
     // Fatal errors.
     kNoError = 0,
@@ -708,6 +780,7 @@ class EchoCancellation {
     float divergent_filter_fraction;
   };
 
+  // Deprecated. Use GetStatistics on the AudioProcessing interface instead.
   // TODO(ajm): discuss the metrics update period.
   virtual int GetMetrics(Metrics* metrics) = 0;
 
@@ -724,8 +797,9 @@ class EchoCancellation {
   // Note that if there are several clients pulling metrics from
   // |GetDelayMetrics()| during a session the first call from any of them will
   // change to one second aggregation window for all.
-  // TODO(bjornv): Deprecated, remove.
+  // Deprecated. Use GetStatistics on the AudioProcessing interface instead.
   virtual int GetDelayMetrics(int* median, int* std) = 0;
+  // Deprecated. Use GetStatistics on the AudioProcessing interface instead.
   virtual int GetDelayMetrics(int* median, int* std,
                               float* fraction_poor_delays) = 0;