From 7043f9d3717d402137716c86e3ec5c358f8bac93 Mon Sep 17 00:00:00 2001
From: Pierre Lespagnol <pierre.lespagnol@savoirfairelinux.com>
Date: Thu, 29 Aug 2019 16:53:43 -0400
Subject: [PATCH] rtcp: fix jitter convertion
Change-Id: I42b390cccfe5a849be4bffccb014534d0e75d723
---
src/media/video/video_rtp_session.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/media/video/video_rtp_session.cpp b/src/media/video/video_rtp_session.cpp
index 3ea7dd0a89..f90723c214 100644
--- a/src/media/video/video_rtp_session.cpp
+++ b/src/media/video/video_rtp_session.cpp
@@ -350,7 +350,9 @@ VideoRtpSession::checkMediumRCTPInfo(RTCPInfo& rtcpi)
totalJitter += ntohl(it.jitter);
}
rtcpi.packetLoss = nbDropNotNull ? (float)( 100 * totalLost) / (256.0 * nbDropNotNull) : 0;
- rtcpi.jitter = totalJitter / vectSize / 16; // millisecond
+ // Jitter is expressed in timestamp unit -> convert to milliseconds
+ // https://stackoverflow.com/questions/51956520/convert-jitter-from-rtp-timestamp-unit-to-millisseconds
+ rtcpi.jitter = (totalJitter / vectSize / 90000.0f) * 1000;
rtcpi.nb_sample = vectSize;
rtcpi.latency = socketPair_->getLastLatency();
return true;
--
GitLab