Commit 6ba5315a authored by Juliusz Chroboczek's avatar Juliusz Chroboczek

Don't mute clients unless they are suffering packet loss.

Otherwise we never recover from low throughput.
parent aef0c731
...@@ -662,15 +662,33 @@ func updateUpBitrate(up *upConnection) { ...@@ -662,15 +662,33 @@ func updateUpBitrate(up *upConnection) {
for _, l := range local { for _, l := range local {
ms := atomic.LoadUint64(&l.maxBitrate.timestamp) ms := atomic.LoadUint64(&l.maxBitrate.timestamp)
bitrate := atomic.LoadUint64(&l.maxBitrate.bitrate) bitrate := atomic.LoadUint64(&l.maxBitrate.bitrate)
loss := atomic.LoadUint32(&l.loss)
if now < ms || now > ms+5000 || bitrate == 0 { if now < ms || now > ms+5000 || bitrate == 0 {
// no rate information
l.setMuted(false) l.setMuted(false)
continue continue
} }
if bitrate < 9600 ||
(l.track.Kind() == webrtc.RTPCodecTypeVideo && isvideo := l.track.Kind() == webrtc.RTPCodecTypeVideo
bitrate < 128000) { minrate1 := uint64(9600)
l.setMuted(true) minrate2 := uint64(19200)
continue if isvideo {
minrate1 = 256000
minrate2 = 512000
}
if bitrate < minrate2 {
if loss <= 13 {
// less than 10% loss, go ahead
bitrate = minrate2
} else if loss <= 64 || !isvideo {
if bitrate < minrate1 {
bitrate = minrate1
}
} else {
// video track with dramatic loss
l.setMuted(true)
continue
}
} }
l.setMuted(false) l.setMuted(false)
if track.maxBitrate > bitrate { if track.maxBitrate > bitrate {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment