From 30ecda146a7e456d7894fa09d7d558d1600e3386 Mon Sep 17 00:00:00 2001 From: "marpan@google.com" Date: Fri, 9 Sep 2011 17:15:12 +0000 Subject: [PATCH] media_opt_util: Added comment and lowered window size parameter. Review URL: http://webrtc-codereview.appspot.com/135018 git-svn-id: http://webrtc.googlecode.com/svn/trunk@575 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/modules/video_coding/main/source/media_opt_util.h | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/modules/video_coding/main/source/media_opt_util.h b/src/modules/video_coding/main/source/media_opt_util.h index 8a02aaee89..361194464e 100644 --- a/src/modules/video_coding/main/source/media_opt_util.h +++ b/src/modules/video_coding/main/source/media_opt_util.h @@ -26,8 +26,12 @@ namespace webrtc { class ListWrapper; -enum { kLossPrHistorySize = 30 }; // 30 time periods -// 1000 ms, total filter length is 30 000 ms +// Number of time periods used for (max) window filter for packet loss +// TODO (marpan): set reasonable window size for filtered packet loss, +// adjustment should be based on logged/real data of loss stats/correlation. +enum { kLossPrHistorySize = 10 }; + +// 1000 ms, total filter length is (kLossPrHistorySize * 1000) ms enum { kLossPrShortFilterWinMs = 1000 }; // Thresholds for hybrid NACK/FEC