better documentation
authorFelix Ableitner <me@nutomic.com>
Tue, 9 Oct 2018 23:22:35 +0000 (18:22 -0500)
committerChocobozzz <me@florianbigard.com>
Wed, 17 Oct 2018 08:05:17 +0000 (10:05 +0200)
shared/models/videos/video-resolution.enum.ts

index 13c0fe9a74d3dfe821c0349bc8258d8a147761e6..4d2644cc0b99cb3edf3c48b8c88c8b89673bd09f 100644 (file)
@@ -49,14 +49,16 @@ export function getTargetBitrate (resolution: VideoResolution, fps: number,
   const baseBitrate = getBaseBitrate(resolution)
   // The maximum bitrate, used when fps === VideoTranscodingFPS.MAX
   // Based on numbers from Youtube, 60 fps bitrate divided by 30 fps bitrate:
-  // 2600 / 1750 = 1.48571428571
-  // 4400 / 3300 = 1.33333333333
+  //  720p: 2600 / 1750 = 1.48571428571
+  // 1080p: 4400 / 3300 = 1.33333333333
   const maxBitrate = baseBitrate * 1.4
   const maxBitrateDifference = maxBitrate - baseBitrate
   const maxFpsDifference = fpsTranscodingConstants.MAX - fpsTranscodingConstants.AVERAGE
   // For 1080p video with default settings, this results in the following formula:
   // 3300 + (x - 30) * (1320/30)
-  // Example outputs: 1080p30: 3300 kbps, 1080p60: 4620 kbps, 720p30: 1750, 720p60: 2450
+  // Example outputs:
+  // 1080p10: 2420 kbps, 1080p30: 3300 kbps, 1080p60: 4620 kbps
+  //  720p10: 1283 kbps,  720p30: 1750 kbps,  720p60: 2450
   return baseBitrate + (fps - fpsTranscodingConstants.AVERAGE) * (maxBitrateDifference / maxFpsDifference)
 }