Hi Community,

Greetings from me! Very sorry for interrupting again!

I find the calculation of "Mean Jitter" KPI for RTP flow is like this (
https://gitlab.com/wireshark/wireshark/-/blob/master/ui/tap-rtp-analysis.c#L497
):

statinfo->mean_jitter = (statinfo->mean_jitter*(statinfo->total_nr-1) +
current_diff) / statinfo->total_nr;

But in my humble opinion, maybe we should use current_jitter instead of
current_diff? Like following one:

statinfo->mean_jitter = (statinfo->mean_jitter*(statinfo->total_nr-1) +
current_jitter) / statinfo->total_nr;

Actually, I submitted an issue in gitlab before (
https://gitlab.com/wireshark/wireshark/-/issues/17600), but unfortunately,
it seems no one noticed it. So very sorry for reposting here, and hope
someone can help to clarify it.

Thanks very much!

Best Regards
Nan Xiao
___________________________________________________________________________
Sent via:    Wireshark-users mailing list <wireshark-users@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-users
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-users
             mailto:wireshark-users-requ...@wireshark.org?subject=unsubscribe

Reply via email to