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

Side by Side Diff: webrtc/modules/congestion_controller/trendline_estimator.h

Issue 2633293004: Improve computational performance of BWE by switching list to deque. (Closed)
Patch Set: Created 3 years, 11 months 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 #ifndef WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_ 10 #ifndef WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_
11 #define WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_ 11 #define WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_
12 12
13 #include <stddef.h> 13 #include <stddef.h>
14 #include <stdint.h> 14 #include <stdint.h>
15 15
16 #include <list> 16 #include <deque>
17 #include <utility> 17 #include <utility>
18 18
19 #include "webrtc/base/constructormagic.h" 19 #include "webrtc/base/constructormagic.h"
20 20
21 namespace webrtc { 21 namespace webrtc {
22 22
23 class TrendlineEstimator { 23 class TrendlineEstimator {
24 public: 24 public:
25 // |window_size| is the number of points required to compute a trend line. 25 // |window_size| is the number of points required to compute a trend line.
26 // |smoothing_coef| controls how much we smooth out the delay before fitting 26 // |smoothing_coef| controls how much we smooth out the delay before fitting
(...skipping 27 matching lines...) Expand all
54 const double smoothing_coef_; 54 const double smoothing_coef_;
55 const double threshold_gain_; 55 const double threshold_gain_;
56 // Used by the existing threshold. 56 // Used by the existing threshold.
57 unsigned int num_of_deltas_; 57 unsigned int num_of_deltas_;
58 // Keep the arrival times small by using the change from the first packet. 58 // Keep the arrival times small by using the change from the first packet.
59 int64_t first_arrival_time_ms; 59 int64_t first_arrival_time_ms;
60 // Exponential backoff filtering. 60 // Exponential backoff filtering.
61 double accumulated_delay_; 61 double accumulated_delay_;
62 double smoothed_delay_; 62 double smoothed_delay_;
63 // Linear least squares regression. 63 // Linear least squares regression.
64 std::list<std::pair<double, double>> delay_hist_; 64 std::deque<std::pair<double, double>> delay_hist_;
65 double trendline_; 65 double trendline_;
66 66
67 RTC_DISALLOW_COPY_AND_ASSIGN(TrendlineEstimator); 67 RTC_DISALLOW_COPY_AND_ASSIGN(TrendlineEstimator);
68 }; 68 };
69 } // namespace webrtc 69 } // namespace webrtc
70 70
71 #endif // WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_ 71 #endif // WEBRTC_MODULES_CONGESTION_CONTROLLER_TRENDLINE_ESTIMATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698