Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1159)

Unified Diff: webrtc/modules/congestion_controller/median_slope_estimator.h

Issue 2512693002: Implement Theil-Sen's method for fitting a line to noisy data (used in bandwidth estimation). (Closed)
Patch Set: Remove PercentileFilter::Clear since no longer used. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/congestion_controller/median_slope_estimator.h
diff --git a/webrtc/modules/congestion_controller/median_slope_estimator.h b/webrtc/modules/congestion_controller/median_slope_estimator.h
new file mode 100644
index 0000000000000000000000000000000000000000..26d7f6144d76b350ce3bbc9589ba09db840eb73e
--- /dev/null
+++ b/webrtc/modules/congestion_controller/median_slope_estimator.h
@@ -0,0 +1,71 @@
+/*
+ * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+#ifndef WEBRTC_MODULES_CONGESTION_CONTROLLER_MEDIAN_SLOPE_ESTIMATOR_H_
+#define WEBRTC_MODULES_CONGESTION_CONTROLLER_MEDIAN_SLOPE_ESTIMATOR_H_
+
+#include <list>
+#include <utility>
+#include <vector>
+
+#include "webrtc/base/analytics/percentile_filter.h"
+#include "webrtc/base/constructormagic.h"
+#include "webrtc/common_types.h"
+
+namespace webrtc {
+
+class MedianSlopeEstimator {
+ public:
+ // |window_size| is the number of points required to compute a trend line.
+ // |threshold_gain| is used to scale the trendline slope for comparison to
+ // the old threshold. Once the old estimator has been removed (or the
+ // thresholds been merged into the estimators), we can just set the
+ // threshold instead of setting a gain.
+ MedianSlopeEstimator(size_t window_size, double threshold_gain);
+ ~MedianSlopeEstimator();
+
+ // Update the estimator with a new sample. The deltas should represent deltas
+ // between timestamp groups as defined by the InterArrival class.
+ void Update(double recv_delta_ms, double send_delta_ms, double now_ms);
+
+ // Returns the estimated trend k multiplied by some gain.
+ // 0 < k < 1 -> the delay increases, queues are filling up
+ // k == 0 -> the delay does not change
+ // k < 0 -> the delay decreases, queues are being emptied
+ double trendline_slope() const { return trendline_ * threshold_gain_; }
+
+ // Returns the number of deltas which the current estimator state is based on.
+ unsigned int num_of_deltas() const { return num_of_deltas_; }
+
+ private:
+ struct DelayInfo {
+ DelayInfo(double time, double delay, size_t slope_count)
+ : time(time), delay(delay) {
+ slopes.reserve(slope_count);
+ }
+ double time;
+ double delay;
+ std::vector<double> slopes;
+ };
+ // Parameters.
+ const size_t window_size_;
+ const double threshold_gain_;
+ // Used by the existing threshold.
+ unsigned int num_of_deltas_;
+ // Theil-Sen robust line fitting
+ double accumulated_delay_;
+ std::list<DelayInfo> delay_hist_;
+ PercentileFilter<double> median_filter_;
+ double trendline_;
+
+ RTC_DISALLOW_COPY_AND_ASSIGN(MedianSlopeEstimator);
+};
+} // namespace webrtc
+
+#endif // WEBRTC_MODULES_CONGESTION_CONTROLLER_MEDIAN_SLOPE_ESTIMATOR_H_

Powered by Google App Engine
This is Rietveld 408576698