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

Issue 1237303002: Modified Simulation Framework Jitter Model (Closed)

Created:
5 years, 5 months ago by magalhaesc
Modified:
5 years, 4 months ago
Reviewers:
*stefan-webrtc
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, mflodman
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Modified Simulation Framework Jitter Model. Using a right-sided (absolute value), truncated gaussian distribution originally with zero mean. Currently truncated at x = 3 * std_dev. Added expected value computation. Modified jitter unittests accordingly. BUG=webrtc:4848 R=stefan@webrtc.org Committed: https://chromium.googlesource.com/external/webrtc/+/bf40b42af585128a5b22299bdfaff706659774bd

Patch Set 1 #

Patch Set 2 : Updated Jitter expected value to suit the truncated gaussian #

Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -26 lines) Patch
M webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimators_test.cc View 4 chunks +5 lines, -5 lines 0 comments Download
M webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.h View 1 chunk +4 lines, -1 line 0 comments Download
M webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework.cc View 1 2 chunks +47 lines, -11 lines 0 comments Download
M webrtc/modules/remote_bitrate_estimator/test/bwe_test_framework_unittest.cc View 3 chunks +14 lines, -9 lines 0 comments Download
M webrtc/modules/remote_bitrate_estimator/test/packet.h View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
magalhaesc
Modified Jitter Model
5 years, 5 months ago (2015-07-15 14:41:55 UTC) #3
stefan-webrtc
On 2015/07/15 14:41:55, magalhaesc wrote: > Modified Jitter Model Update the BUG= to refer to ...
5 years, 5 months ago (2015-07-15 15:06:30 UTC) #4
magalhaesc
Updated Jitter expected value to suit the truncated gaussian
5 years, 5 months ago (2015-07-15 15:34:24 UTC) #5
magalhaesc
Updated Jitter expected value to suit the truncated gaussian
5 years, 5 months ago (2015-07-15 15:35:18 UTC) #6
magalhaesc
5 years, 5 months ago (2015-07-15 17:47:27 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
bf40b42af585128a5b22299bdfaff706659774bd (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698