X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fhelpers%2Fffmpeg-utils.ts;h=7c45f36326de30ba57b6d143d8e8258d9c2e8623;hb=8b60488020883c66d3831eacd030893ab184268f;hp=eb1c86ab96a11a8a613357ba11cebaac51dd148c;hpb=4176e227cb18c40e13f30f4634d128cc3169e9d4;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/helpers/ffmpeg-utils.ts b/server/helpers/ffmpeg-utils.ts index eb1c86ab9..7c45f3632 100644 --- a/server/helpers/ffmpeg-utils.ts +++ b/server/helpers/ffmpeg-utils.ts @@ -1,11 +1,33 @@ import * as ffmpeg from 'fluent-ffmpeg' import { join } from 'path' import { VideoResolution } from '../../shared/models/videos' -import { CONFIG, VIDEO_TRANSCODING_FPS } from '../initializers' -import { unlinkPromise } from './core-utils' +import { CONFIG, FFMPEG_NICE, VIDEO_TRANSCODING_FPS } from '../initializers' import { processImage } from './image-utils' import { logger } from './logger' import { checkFFmpegEncoders } from '../initializers/checker' +import { remove } from 'fs-extra' + +function computeResolutionsToTranscode (videoFileHeight: number) { + const resolutionsEnabled: number[] = [] + const configResolutions = CONFIG.TRANSCODING.RESOLUTIONS + + // Put in the order we want to proceed jobs + const resolutions = [ + VideoResolution.H_480P, + VideoResolution.H_360P, + VideoResolution.H_720P, + VideoResolution.H_240P, + VideoResolution.H_1080P + ] + + for (const resolution of resolutions) { + if (configResolutions[ resolution + 'p' ] === true && videoFileHeight > resolution) { + resolutionsEnabled.push(resolution) + } + } + + return resolutionsEnabled +} async function getVideoFileResolution (path: string) { const videoStream = await getVideoFileStream(path) @@ -56,7 +78,7 @@ async function generateImageFromVideoFile (fromPath: string, folder: string, ima try { await new Promise((res, rej) => { - ffmpeg(fromPath) + ffmpeg(fromPath, { niceness: FFMPEG_NICE.THUMBNAIL }) .on('error', rej) .on('end', () => res(imageName)) .thumbnail(options) @@ -68,7 +90,7 @@ async function generateImageFromVideoFile (fromPath: string, folder: string, ima logger.error('Cannot generate image from video %s.', fromPath, { err }) try { - await unlinkPromise(pendingImagePath) + await remove(pendingImagePath) } catch (err) { logger.debug('Cannot remove pending image path after generation error.', { err }) } @@ -84,11 +106,15 @@ type TranscodeOptions = { function transcode (options: TranscodeOptions) { return new Promise(async (res, rej) => { - let command = ffmpeg(options.inputPath) + let command = ffmpeg(options.inputPath, { niceness: FFMPEG_NICE.TRANSCODING }) .output(options.outputPath) - .outputOption('-threads ' + CONFIG.TRANSCODING.THREADS) .preset(standard) + if (CONFIG.TRANSCODING.THREADS > 0) { + // if we don't set any threads ffmpeg will chose automatically + command = command.outputOption('-threads ' + CONFIG.TRANSCODING.THREADS) + } + let fps = await getVideoFileFPS(options.inputPath) if (options.resolution !== undefined) { // '?x720' or '720x?' for example @@ -129,7 +155,9 @@ export { getDurationFromVideoFile, generateImageFromVideoFile, transcode, - getVideoFileFPS + getVideoFileFPS, + computeResolutionsToTranscode, + audio } // --------------------------------------------------------------------------- @@ -154,8 +182,8 @@ function getVideoFileStream (path: string) { * and quality. Superfast and ultrafast will give you better * performance, but then quality is noticeably worse. */ -function veryfast (ffmpeg) { - ffmpeg +function veryfast (_ffmpeg) { + _ffmpeg .preset(standard) .outputOption('-preset:v veryfast') .outputOption(['--aq-mode=2', '--aq-strength=1.3']) @@ -175,8 +203,8 @@ function veryfast (ffmpeg) { /** * A preset optimised for a stillimage audio video */ -function audio (ffmpeg) { - ffmpeg +function audio (_ffmpeg) { + _ffmpeg .preset(veryfast) .outputOption('-tune stillimage') } @@ -185,25 +213,36 @@ function audio (ffmpeg) { * A toolbox to play with audio */ namespace audio { - export const get = (ffmpeg, pos = 0) => { + export const get = (_ffmpeg, pos: number | string = 0) => { // without position, ffprobe considers the last input only // we make it consider the first input only - ffmpeg - .ffprobe(pos, (_,data) => { - return data['streams'].find(stream => { - return stream['codec_type'] === 'audio' - }) + // if you pass a file path to pos, then ffprobe acts on that file directly + return new Promise<{ absolutePath: string, audioStream?: any }>((res, rej) => { + _ffmpeg.ffprobe(pos, (err,data) => { + if (err) return rej(err) + + if ('streams' in data) { + const audioStream = data['streams'].find(stream => stream['codec_type'] === 'audio') + if (audioStream) { + return res({ + absolutePath: data.format.filename, + audioStream + }) + } + } + return res({ absolutePath: data.format.filename }) }) + }) } export namespace bitrate { - export const baseKbitrate = 384 + const baseKbitrate = 384 const toBits = (kbits: number): number => { return kbits * 8000 } export const aac = (bitrate: number): number => { switch (true) { - case bitrate > toBits(384): + case bitrate > toBits(baseKbitrate): return baseKbitrate default: return -1 // we interpret it as a signal to copy the audio stream as is @@ -211,6 +250,11 @@ namespace audio { } export const mp3 = (bitrate: number): number => { + /* + a 192kbit/sec mp3 doesn't hold as much information as a 192kbit/sec aac. + That's why, when using aac, we can go to lower kbit/sec. The equivalences + made here are not made to be accurate, especially with good mp3 encoders. + */ switch (true) { case bitrate <= toBits(192): return 128 @@ -229,34 +273,39 @@ namespace audio { * As for the audio, quality '5' is the highest and ensures 96-112kbps/channel * See https://trac.ffmpeg.org/wiki/Encode/AAC#fdk_vbr */ -async function standard (ffmpeg) { - let _bitrate = audio.bitrate.baseKbitrate - let _ffmpeg = ffmpeg +async function standard (_ffmpeg) { + let localFfmpeg = _ffmpeg .format('mp4') .videoCodec('libx264') .outputOption('-level 3.1') // 3.1 is the minimal ressource allocation for our highest supported resolution .outputOption('-b_strategy 1') // NOTE: b-strategy 1 - heuristic algorythm, 16 is optimal B-frames for it .outputOption('-bf 16') // NOTE: Why 16: https://github.com/Chocobozzz/PeerTube/pull/774. b-strategy 2 -> B-frames<16 + .outputOption('-map_metadata -1') // strip all metadata .outputOption('-movflags faststart') - let _audio = audio.get(_ffmpeg) + const _audio = await audio.get(localFfmpeg) - if (!_audio) return _ffmpeg.noAudio() - - // we try to reduce the ceiling bitrate by making rough correspondances of bitrates - // of course this is far from perfect, but it might save some space in the end - if (audio.bitrate[_audio['codec_name']]) { - _bitrate = audio.bitrate[_audio['codec_name']](_audio['bit_rate']) - if (_bitrate === -1) { - return _ffmpeg.audioCodec('copy') - } + if (!_audio.audioStream) { + return localFfmpeg.noAudio() } // we favor VBR, if a good AAC encoder is available if ((await checkFFmpegEncoders()).get('libfdk_aac')) { - return _ffmpeg + return localFfmpeg .audioCodec('libfdk_aac') .audioQuality(5) } - return _ffmpeg.audioBitrate(_bitrate) + // we try to reduce the ceiling bitrate by making rough correspondances of bitrates + // of course this is far from perfect, but it might save some space in the end + const audioCodecName = _audio.audioStream['codec_name'] + let bitrate: number + if (audio.bitrate[audioCodecName]) { + bitrate = audio.bitrate[audioCodecName](_audio.audioStream['bit_rate']) + + if (bitrate === -1) return localFfmpeg.audioCodec('copy') + } + + if (bitrate !== undefined) return localFfmpeg.audioBitrate(bitrate) + + return localFfmpeg }