Reland "Tune vp9 screenshare bitrate and framerate of spatial layers"
This is a reland after changes to the downstream project VP9 screenshare is not used currently, and with these values according to local testing with screenshare_loopback, we get performance not worse than current vp8 settings for similar uplink and downlink values. Original Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/126226 Bug: webrtc:10257 Change-Id: Ib21d7678bd839a3c47457515b0d768c0b979ea40 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/126524 Reviewed-by: Johannes Kron <kron@webrtc.org> Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org> Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org> Cr-Commit-Position: refs/heads/master@{#27040}
This commit is contained in:
committed by
Commit Bot
parent
62c7b39c71
commit
cad95b831e
@ -23,10 +23,10 @@ namespace {
|
||||
const size_t kMinVp9SvcBitrateKbps = 30;
|
||||
|
||||
const size_t kMaxNumLayersForScreenSharing = 3;
|
||||
const float kMaxScreenSharingLayerFramerateFps[] = {5.0, 5.0, 30.0};
|
||||
const size_t kMinScreenSharingLayerBitrateKbps[] = {30, 150, 500};
|
||||
const size_t kTargetScreenSharingLayerBitrateKbps[] = {150, 350, 1000};
|
||||
const size_t kMaxScreenSharingLayerBitrateKbps[] = {200, 500, 1000};
|
||||
const float kMaxScreenSharingLayerFramerateFps[] = {5.0, 10.0, 30.0};
|
||||
const size_t kMinScreenSharingLayerBitrateKbps[] = {30, 200, 500};
|
||||
const size_t kTargetScreenSharingLayerBitrateKbps[] = {150, 350, 950};
|
||||
const size_t kMaxScreenSharingLayerBitrateKbps[] = {250, 500, 950};
|
||||
|
||||
} // namespace
|
||||
|
||||
|
||||
@ -54,7 +54,7 @@ TEST(SvcConfig, ScreenSharing) {
|
||||
const SpatialLayer& layer = spatial_layers[i];
|
||||
EXPECT_EQ(layer.width, 1920);
|
||||
EXPECT_EQ(layer.height, 1080);
|
||||
EXPECT_EQ(layer.maxFramerate, (i < 2) ? 5 : 30);
|
||||
EXPECT_EQ(layer.maxFramerate, (i < 1) ? 5 : (i < 2 ? 10 : 30));
|
||||
EXPECT_EQ(layer.numberOfTemporalLayers, 1);
|
||||
EXPECT_LE(layer.minBitrate, layer.maxBitrate);
|
||||
EXPECT_LE(layer.minBitrate, layer.targetBitrate);
|
||||
|
||||
Reference in New Issue
Block a user