Re: [webrtc-stats] Add clarification that inbound-rtp.bytesReceived includes retransmissions. (#755)

If I may quote you @fippo running https://jsfiddle.net/fippo/hvpxf5L0/3/ with and without https://webrtc-review.googlesource.com/c/src/+/295260 which did modify inbound-rtp counters:

> with this patch yields (--force-fieldtrials=WebRtcStatsReportIdl/Disabled/ helps)
> SENT outbound-rtp p=58066 h=5980 packetsSent=98 retransmittedPacketsSent=0
> SENT transport p=66422 h=undefined packetsSent=103 retransmittedPacketsSent=undefined
> RECV inbound-rtp p=58066 h=5980 packetsReceived=98 retransmittedPacketsReceived=21
> RECV transport p=66422 h=undefined packetsReceived=103 retransmittedPacketsReceived=undefined
>
> while without it the following (from a different connection)
> SENT outbound-rtp p=83872 h=3536 packetsSent=108 retransmittedPacketsSent=0
> SENT transport p=89814 h=undefined packetsSent=112 retransmittedPacketsSent=undefined
> RECV inbound-rtp p=78714 h=2536 packetsReceived=97 retransmittedPacketsReceived=undefined
> RECV transport p=89814 h=undefined packetsReceived=112 retransmittedPacketsReceived=undefined


-- 
GitHub Notification of comment by henbos
Please view or discuss this issue at https://github.com/w3c/webrtc-stats/issues/755#issuecomment-1532110642 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 2 May 2023 20:32:24 UTC