- From: Varun Singh via GitHub <sysbot+gh@w3.org>
- Date: Wed, 15 Mar 2023 02:26:00 +0000
- To: public-webrtc-logs@w3.org
retransmittedPacketsSent and retransmittedBytesSent are already included in packetsSent and bytesSent. So they should be include retransmissions. Would this definition improve or clarify: >headerBytesSent of type unsigned long long > >Total number of RTP header and padding bytes sent for this [SSRC](https://www.w3.org/TR/webrtc-stats/#dfn-ssrc). This does not include the size of transport layer headers such as IP or UDP. **headerBytesSent includes retransmissions**. Hence, headerBytesSent + bytesSent equals the number of bytes sent as payload over the transport. For totalPacketSendDelay says the measurement is added each time the packetSent is incremented. And the packetSent explicitly states that it includes retransmissions. Would the following clarification help? >totalPacketSendDelay of type double > >The total number of seconds that packets have spent buffered locally before being transmitted onto the network. The time is measured from when a packet is emitted from the RTP packetizer until it is handed over to the OS network socket. This measurement is added to [totalPacketSendDelay](https://www.w3.org/TR/webrtc-stats/#dom-rtcoutboundrtpstreamstats-totalpacketsenddelay) when [packetsSent](https://www.w3.org/TR/webrtc-stats/#dom-rtcsentrtpstreamstats-packetssent) is incremented. packetSent includes retransmitted packets, therefore, this measurement should be include send delays for retransmitted packets. -- GitHub Notification of comment by vr000m Please view or discuss this issue at https://github.com/w3c/webrtc-stats/issues/747#issuecomment-1469201364 using your GitHub account -- Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Wednesday, 15 March 2023 02:26:04 UTC