other_video_transcoding 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b33c51af021fb92ec9d41e7b4b95e92740b9c15832f276a471d6ecd8a92abe4
-  data.tar.gz: c3e030f58c77f2cce69cd02a3563428f4a0f6dbed0b6216928f4c40c6b2f3d73
+  metadata.gz: 88e79f218a2cdbb3a930db25fd5d5b94e6294fec90f451bb979457c68ae13a82
+  data.tar.gz: cc6b33de67d3da5fb80d59634181ccdd89aa539cb4d35b5d5729bd3b4ee5dae0
 SHA512:
-  metadata.gz: 5f5e7f4135a53ed651720e0803f605a52768cb12758fcb9c13c15c4d8cc59ec94340f02aab11d9c3db9b390bf296c51b9fb7f49c8314bbf4906352252dbc4942
-  data.tar.gz: 8df50e3a3be13cb47483fd5b0ce9b2c9e97b3b4fe6755e5843e450bf177709df070e82e2ba18ab79c76d5f30172d588b9407b6aec25757ffcb0fd6e45e180881
+  metadata.gz: 638f9905c7df90b4c7921ae6a1d4b54e93f264c119a2599356ce911132b9688eba83edfe5ab2a0d6037326ce0c62c7aa4813792da45e15840f06d38bff1f999c
+  data.tar.gz: e213384ceb4e30a711e0a161ec371e72cb53b0d8d562ee7dc73dd8aade72dca2df1e8855800643dc36afe4cfa4356ae55a40fd80b2de503944261e7042388f73

data/CHANGELOG.md

@@ -2,6 +2,39 @@
 
 This single document contains all of the notes created for each [release](https://github.com/donmelton/other_video_transcoding/releases).
 
+## [0.9.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.9.0)
+
+Saturday, June 5, 2021
+
+Nvidia encoders:
+
+* Add a `--nvenc-recommended` option to `other-transcode` which easily and optimally configures the Nvidia H.264 and HEVC video encoders, equivalent to using:
+    * `--nvenc-spatial-aq --nvenc-lookahead 32 --nvenc-bframe-refs middle`
+* The `--nvenc-bframe-refs` option is also new and using `middle` as its argument increases compression efficiency. However, it's not supported on all Nvidia hardware. Which means, neither is the `--nvenc-recommended` option.
+* So, using `--nvenc-recommended` for H.264 encoding requires at minimum a fourth generation Pascal-based GPU like the GTX 1080. And using it for HEVC encoding requires at minimum a more recent sixth generation Turing-based model like the GTX 1660 or RTX 2070.
+* Add a `--nvenc-cq` option to `other-transcode` to allow constant quality (CQ) ratecontrol. This option ignores the target video bitrate and takes a single numerical argument between `1` and `51` with lower values indicating higher quality. However, only values between `25` and `30` are really practical for 1080p content. It's recommended to start with `--nvenc-cq 28` and adjust as needed. Fractional values are also supported.
+* While CQ ratecontrol can adapt better to extremely dynamic content compared to the default average bitrate (ABR) ratecontrol system, the output bitrates produced by CQ can vary significantly and it can be slightly more prone to color banding with the H.264 encoder. So, for now, consider the `--nvenc-cq` option an experimental feature.
+* Deprecate the `--nvenc-rc-mode` option in `other-transcode` because maximum rate and buffer size values are already specified so an explicit ratecontrol mode is not required.
+
+Audio:
+
+* Modify `other-transcode` to no longer explicitly set audio bitrates and instead rely on the defaults from the audio encoders included with `ffmpeg`. This is being done to both simplify the code in `other-transcode` and to leverage the judgement of the audio encoder developers, which means:
+    * AC-3 surround audio is lowered from 640 Kbps to 448 Kbps.
+    * Dolby Digital Plus (Enhanced AC-3) surround audio is raised from 384 Kbps to 448 Kbps.
+    * AAC stereo audio is lowered from 256 Kbps to 128 Kbps when using the native `ffmpeg` or Apple AudioToolbox encoders.
+* If desired, use the `--surround-bitrate` and `--stereo-bitrate` options to restore the previous default audio bitrates.
+* Add a `--aac-only` option to `other-transcode` to force transcoding of all audio into AAC format, copying only tracks which are already in that format. Surround tracks are converted to a 5.1-channel AAC format at 341 Kbps when using the native `ffmpeg` encoder. This channel layout is compatible with most playback devices. Allowing more than 5.1 channels would significantly reduce playback compatibility.
+* Deprecate the `--aac-stereo` option in `other-transcode` and replace with a new `--eac3-aac` option, which uses Dolby Digital Plus format only for surround audio with AAC format for stereo audio. Since the `--aac-stereo` option only made sense when used in combination with the `--eac3` option, this makes `--eac3-aac` more convenient.
+
+Other changes:
+
+* Change the default ratecontrol maximum rate and buffer size values in `other-transcode`, as well as how those values increased when the target video bitrate is increased. The default values now also differ depending on whether the output video is in H.264 or HEVC format.
+* Modify the `--rc-maxrate` and `--rc-bufsize` options in `other-transcode` to allow specific bitrates as arguments and not just multiples of the video bitrate target. These options are most useful for lowering maximum rate and buffer size values. To raise those values, increase the target video bitrate instead.
+* Change the pixel format and force a Main 10 video profile in `other-transcode` when using the Apple VideoToolbox encoder to create output with a 10-bit color depth. Correct generation of 10-bit output requires macOS Big Sur or later.
+* Modify `other-transcode` to copy the three basic color properties (primaries, transer and space) from the input to the output when those properties are availabe. Otherwise use some sensible defaults.
+* Modify `other-transcode` to force a H.264 level when using `--x264` with slower presets. This ensures that the `x264` encoder continues to generate a video stream compatible with most playback devices when those presets are used.
+* Modify `other-transcode` to remove the hack which avoids using the Matroksa muxer `-disposition` option in old versions of `ffmpeg`. This means version 4.3 or later of `ffmpeg` is now required to run `other-transcode`.
+
 ## [0.8.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.8.0)
 
 Saturday, February 13, 2021
@@ -59,7 +92,7 @@ Tuesday, December 22, 2020
 * Lower the default target bitrates for 8-bit HEVC video in `other-transcode` to match the defaults for 10-bit HEVC video. This means, for example, the default target for HEVC at a 1080p resolution will be 6000 Kbps no matter the output bit depth.
 * Modify `other-transcode` to set the video buffer size equal to the maximum video bitrate when using an Nvidia encoder, essentially adding `--rc-bufsize 3` to the command line. Previously the buffer size was never explicitly set so `ffmpeg` would use a default value of twice the target bitrate. Since the maximum bitrate is normally three times the target bitrate this meant the buffer size was actually smaller than the maximum. While this didn't cause any known problems, Nvidia recommends a larger buffer size to improve quality. However, using `--rc-bufsize 0` will restore the old behavior and the default value from `ffmpeg`.
 * Ignore the `--nvenc-lookahead` option in `other-transcode` when the argument is `0` since such a value won't change the behavior of an Nvidia encoder anyway.
-* Add a `--limit-ac3-surround` option to `other-transcode` which prevents surround audio in AC-3 or Dolby Digital Plus (Enhanced AC-3) format from being copied instead of transcoded when the orginal bitrate is above the transcoding bitrate. This allows setting a lower target with the `--surround-bitrate` option in order to force higher-bitrate tracks to be transcoded instead of copied. 
+* Add a `--limit-ac3-surround` option to `other-transcode` which prevents surround audio in AC-3 or Dolby Digital Plus (Enhanced AC-3) format from being copied instead of transcoded when the orginal bitrate is above the transcoding bitrate. This allows setting a lower target with the `--surround-bitrate` option in order to force higher-bitrate tracks to be transcoded instead of copied.
 * Reduce the minimum bitrates for Dolby Digital Plus audio in `other-transcode` from 256, 128 and 64 Kbps for surround, stereo and mono layouts to 192, 96 and 48 Kbps. The default bitrates for Dolby Digital Plus audio remain the same and this change does not affect audio output in AC-3 or AAC formats.
 
 ## [0.5.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.5.0)

data/bin/ask-ffmpeg-log

@@ -16,7 +16,7 @@ module Transcoding
   class Command
     def about
       <<-HERE
-ask-ffmpeg-log 0.8.0
+ask-ffmpeg-log 0.9.0
 Copyright (c) 2019-2021 Don Melton
       HERE
     end

data/bin/other-transcode

@@ -18,7 +18,7 @@ module Transcoding
   class Command
     def about
       <<-HERE
-other-transcode 0.8.0
+other-transcode 0.9.0
 Copyright (c) 2019-2021 Don Melton
       HERE
     end
@@ -139,15 +139,21 @@ Video options:
       <<-HERE
     --yadif-params KEY=VALUE[:KEY=VALUE]...
                     override yadif deinterlace filter configuration
-    --rc-maxrate FACTOR, --rc-bufsize FACTOR
+    --rc-maxrate FACTOR, --rc-bufsize FACTOR|BITRATE
                     set ratecontrol maximum rate and/or buffer size
-                      as multiple of video bitrate target
+                      as multiple of video bitrate target or specific bitrate
     --copy-video    disable transcoding and copy original video track
 
 Apple Video Toolbox encoder options:
     --vt-allow-sw   allow software encoding
 
 Nvidia video encoder options:
+    --nvenc-recommended
+                    apply optimal quality settings, equivalent to using:
+                      `--nvenc-spatial-aq`
+                      `--nvenc-lookahead 32`
+                      `--nvenc-bframe-refs middle`
+                      (not supported on all Nvidia hardware)
     --nvenc-spatial-aq
                     enable spatial adaptive quantization (AQ)
     --nvenc-temporal-aq
@@ -160,8 +166,10 @@ Nvidia video encoder options:
                     set number of reference frames
     --nvenc-bframes FRAMES
                     set maximum number of B-frames
-    --nvenc-rc-mode vbr|vbr_hq
-                    set ratecontrol mode (default: vbr)
+    --nvenc-bframe-refs each|middle
+                    set mode for using B-frames as reference frames
+    --nvenc-cq QUALITY
+                    use constant quality (CQ) ratecontrol at target quality
 
 Intel Quick Sync video encoder options:
     --qsv-refs FRAMES
@@ -241,35 +249,39 @@ Audio options:
     def usage15
       <<-HERE
     --surround-bitrate BITRATE
-                    set surround audio bitrate (default: 640)
+                    set surround audio bitrate (default: 448)
     --stereo-bitrate BITRATE
-                    set stereo audio bitrate (default: 256)
+                    set stereo audio bitrate (default: 128)
       HERE
     end
 
     def usage16
       <<-HERE
     --mono-bitrate BITRATE
-                    set mono audio bitrate (default: 50% of stereo bitrate)
+                    set mono audio bitrate (default: ~50% of stereo bitrate)
       HERE
     end
 
     def usage17
       <<-HERE
     --eac3          use Dolby Digital Plus (E-AC-3) format for all audio
-                      (default bitrates: 384 for surround, 192 for stereo)
+                      (default bitrates: 448 for surround, 192 for stereo)
       HERE
     end
 
     def usage18
       <<-HERE
-    --aac-stereo    use AAC format for stereo audio
+    --eac3-aac      use Dolby Digital Plus format only for surround audio
+                      with AAC format for stereo audio
+    --aac-only      use AAC format for all audio,
+                      disabling passthrough of audio in other formats
+                      (default bitrates: 341 for surround, 128 for stereo)
     --limit-ac3-surround
                     don't copy surround audio in AC-3 format
-                      when orginal bitrate is above transcoding bitrate
+                      when orginal bitrate is above passthrough bitrate
     --keep-ac3-stereo
                     copy stereo and mono audio in AC-3 format
-                      even when orginal bitrate is above transcoding bitrate
+                      even when orginal bitrate is above passthrough bitrate
     --pass-dts      enable passthrough of audio in DTS and DTS-ES formats
       HERE
     end
@@ -311,7 +323,6 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       @detect = false
       @preview = false
       @format = :mkv
-      @mkv_options = []
       @copy_track_names = false
       @max_muxing_queue_size = nil
       @dry_run = false
@@ -345,7 +356,8 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       @nvenc_multipass = nil
       @nvenc_refs = nil
       @nvenc_bframes = nil
-      @nvenc_rc_mode = 'vbr'
+      @nvenc_bframe_refs = nil
+      @nvenc_cq = nil
       @qsv_refs = nil
       @qsv_bframes = nil
       @amf_quality = nil
@@ -370,6 +382,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       @mono_bitrate = nil
       @surround_encoder = 'ac3'
       @stereo_encoder = nil
+      @aac_fallback_encoder = 'aac'
       @keep_ac3_surround = true
       @keep_ac3_stereo = false
       @pass_dts = false
@@ -566,6 +579,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
           @target = [arg.to_i, 1].max
         end
 
+        @nvenc_cq = nil
         @encoder = nil if @encoder == 'copy'
       end
 
@@ -672,6 +686,13 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         @vt_allow_sw = true
       end
 
+      opts.on '--nvenc-recommended' do
+        @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
+        @nvenc_spatial_aq = true
+        @nvenc_lookahead = 32
+        @nvenc_bframe_refs = 'middle'
+      end
+
       opts.on '--nvenc-spatial-aq' do
         @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
         @nvenc_spatial_aq = true
@@ -713,17 +734,30 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         @nvenc_bframes = [[arg, 0].max, 4].min
       end
 
-      opts.on '--nvenc-rc-mode ARG' do |arg|
+      opts.on '--nvenc-bframe-refs ARG' do |arg|
         @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
 
-        @nvenc_rc_mode = case arg
-        when 'vbr', 'vbr_hq'
+        @nvenc_bframe_refs = case arg
+        when 'each', 'middle'
           arg
         else
-          fail UsageError, "invalid rate control argument: #{arg}"
+          fail UsageError, "invalid B-frames as references argument: #{arg}"
         end
       end
 
+      opts.on '--nvenc-cq ARG', Float do |arg|
+        @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
+        @nvenc_cq = [[arg, 1].max, 51].min.to_s.sub(/\.0$/, '')
+      end
+
+      opts.on '--nvenc-rc-mode ARG' do |arg|
+        Kernel.warn '**********'
+        Kernel.warn 'Using deprecated option: --nvenc-rc-mode'
+        Kernel.warn '**********'
+        @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
+        fail UsageError, "invalid rate control argument: #{arg}" unless arg =~ /^vbr(_hq)?$/
+      end
+
       opts.on '--qsv-refs ARG', Integer do |arg|
         @encoder = @hevc ? 'hevc_qsv' : 'h264_qsv'
         @qsv_refs = [arg, 0].max
@@ -860,6 +894,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
 
       opts.on '--stereo-bitrate ARG', Integer do |arg|
         @stereo_bitrate = arg
+        @mono_bitrate ||= @stereo_bitrate / 2
       end
 
       opts.on '--mono-bitrate ARG', Integer do |arg|
@@ -871,20 +906,35 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         @stereo_encoder   = 'eac3'
       end
 
+      opts.on '--eac3-aac' do
+        @surround_encoder = 'eac3'
+        @stereo_encoder = nil
+      end
+
       opts.on '--aac-stereo' do
+        Kernel.warn '**********'
+        Kernel.warn 'Using deprecated option: --aac-stereo'
+        Kernel.warn '**********'
         @stereo_encoder = nil
       end
 
+      opts.on '--aac-only' do
+        @surround_encoder = nil
+        @stereo_encoder   = nil
+      end
+
       opts.on '--limit-ac3-surround' do
         @keep_ac3_surround = false
       end
 
       opts.on '--keep-ac3-stereo' do
         @keep_ac3_stereo = true
+        @surround_encoder ||= 'ac3'
       end
 
       opts.on '--pass-dts' do
         @pass_dts = true
+        @surround_encoder ||= 'ac3'
       end
 
       opts.on '--add-subtitle ARG' do |arg|
@@ -938,32 +988,6 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       @audio_selections.uniq!
       @subtitle_selections.uniq!
 
-      if @surround_encoder == 'eac3'
-        @surround_bitrate ||= 384
-        @surround_bitrate = [[@surround_bitrate, 192].max, 768].min
-      else
-        @surround_bitrate ||= 640
-        @surround_bitrate = [[@surround_bitrate, 256].max, 640].min
-      end
-
-      if @stereo_encoder == 'eac3'
-        @stereo_bitrate ||= 192
-        @stereo_bitrate = [[@stereo_bitrate, 96].max, 768].min
-      else
-        @stereo_bitrate ||= 256
-        @stereo_bitrate = [[@stereo_bitrate, 128].max, 320].min
-      end
-
-      if @mono_bitrate.nil?
-        @mono_bitrate = @stereo_bitrate / 2
-      else
-        if @stereo_encoder == 'eac3'
-          @mono_bitrate = [[@mono_bitrate, 48].max, 768].min
-        else
-          @mono_bitrate = [[@mono_bitrate, 64].max, 256].min
-        end
-      end
-
       [
         ['ffprobe', '-loglevel', 'quiet', '-version'],
         ['ffmpeg', '-loglevel', 'quiet', '-version'],
@@ -1002,6 +1026,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         end
       end
 
+      @nvenc_cq = nil unless @encoder =~ /nvenc$/
       @ten_bit = (@hevc and @encoder =~ /(nvenc|qsv|x265)$/ ? true : false) if @ten_bit.nil?
       @target_2160p ||= @hevc ? 8000 : 12000
       @target_1080p ||= @hevc ? 4000 :  6000
@@ -1012,14 +1037,12 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       if @stereo_encoder.nil?
         if encoders =~ /aac_at/ or encoders =~ /libfdk_aac/
           @stereo_encoder = $MATCH
+          @aac_fallback_encoder = 'libfdk_aac' if encoders =~ /libfdk_aac/
         else
           @stereo_encoder = 'aac'
         end
-      end
 
-      if @format == :mkv
-        capabilities = get_muxer_capabilities
-        @mkv_options = ['-default_mode', 'passthrough'] if capabilities =~ /passthrough/
+        @surround_encoder ||= @stereo_encoder
       end
     end
 
@@ -1069,17 +1092,17 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         IO.popen([
           'ffmpeg',
           '-loglevel', 'quiet',
-          '-nostdin'
-        ] + (encoder =~ /vaapi$/ ? ['-vaapi_device', '/dev/dri/renderD128'] : []) + [
+          '-nostdin',
+          *(encoder =~ /vaapi$/ ? ['-vaapi_device', '/dev/dri/renderD128'] : []),
           '-i', path,
-          '-frames:v', '1'
-        ] + (encoder =~ /vaapi$/ ? ['-filter:v', 'format=nv12,hwupload'] : []) + [
+          '-frames:v', '1',
+          *(encoder =~ /vaapi$/ ? ['-filter:v', 'format=nv12,hwupload'] : []),
           '-c:v', encoder,
-          '-b:v', '1000k'
-        ] + (encoder =~ /nvenc$/ ? ['-rc:v', @nvenc_rc_mode] : []) +
-            (encoder == 'h264_qsv' ? ['-look_ahead:v', '1'] : []) +
-            (encoder == 'hevc_qsv' ? ['-load_plugin:v', 'hevc_hw'] : []) +
-            (encoder =~ /amf$/ ? ['-rc:v', 'vbr_latency'] : []) + [
+          '-b:v', '1000k',
+          *(encoder =~ /nvenc$/ ? ['-rc:v', 'vbr'] : []),
+          *(encoder == 'h264_qsv' ? ['-look_ahead:v', '1'] : []),
+          *(encoder == 'hevc_qsv' ? ['-load_plugin:v', 'hevc_hw'] : []),
+          *(encoder =~ /amf$/ ? ['-rc:v', 'vbr_latency'] : []),
           '-an',
           '-sn',
           '-ignore_unknown',
@@ -1097,30 +1120,6 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       $CHILD_STATUS.exitstatus == 0
     end
 
-    def get_muxer_capabilities
-      Kernel.warn 'Getting muxer capabilities...'
-      output = ''
-
-      begin
-        IO.popen([
-          'ffmpeg',
-          '-loglevel', 'quiet',
-          '-h', 'muxer=matroska'
-        ], :err=>[:child, :out]) do |io|
-          io.each do |line|
-            Kernel.warn line if @debug
-            output += line
-          end
-        end
-      rescue SystemCallError => e
-        raise "getting muxer capabilities failed: #{e}"
-      end
-
-      fail 'getting muxer capabilities failed' unless $CHILD_STATUS.exitstatus == 0
-
-      output
-    end
-
     def process_input(path)
       seconds = Time.now.tv_sec
 
@@ -1182,16 +1181,16 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       ffmpeg_command = [
         'ffmpeg',
         '-loglevel', (@debug ? 'verbose' : 'error'),
-        '-stats'
-      ] + time_options +
-          decode_options + [
-        '-i', path
-      ] + (@max_muxing_queue_size.nil? ? [] : ['-max_muxing_queue_size', @max_muxing_queue_size.to_s]) +
-          encode_options +
-          get_audio_options(media_info) +
-          get_subtitle_options(media_info, burn_subtitle) + [
-        '-metadata:g', 'title='
-      ] + (@format == :mkv ? @mkv_options : ['-movflags', 'disable_chpl']) + [
+        '-stats',
+        *time_options,
+        *decode_options,
+        '-i', path,
+        *(@max_muxing_queue_size.nil? ? [] : ['-max_muxing_queue_size', @max_muxing_queue_size.to_s]),
+        *encode_options,
+        *get_audio_options(media_info),
+        *get_subtitle_options(media_info, burn_subtitle),
+        '-metadata:g', 'title=',
+        *(@format == :mkv ? ['-default_mode', 'passthrough'] : ['-movflags', 'disable_chpl']),
         output_path
       ]
 
@@ -1351,10 +1350,11 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         index += 1
         puts "\##{index} subtitle:"
         print "      format = #{stream['codec_name']}"
-        frames = stream.fetch('tags', {}).fetch('NUMBER_OF_FRAMES-eng', '')
+        tags = stream.fetch('tags', {})
+        frames = tags.fetch('NUMBER_OF_FRAMES', tags.fetch('NUMBER_OF_FRAMES-eng', ''))
         puts frames.empty? ? '' : " / #{frames} " + (frames == 1 ? 'frame' : 'frames')
-        puts "    language = #{stream.fetch('tags', {}).fetch('language', '')}"
-        title = stream.fetch('tags', {}).fetch('title', '')
+        puts "    language = #{tags.fetch('language', '')}"
+        title = tags.fetch('title', '')
         puts "       title = #{title}" unless title.empty?
         default = (stream['disposition']['default'] == 1)
         forced  = (stream['disposition']['forced'] == 1)
@@ -1369,8 +1369,8 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
     end
 
     def get_bitrate(stream)
-      bitrate = stream.fetch('bit_rate', '')
-      bitrate = stream.fetch('tags', {}).fetch('BPS-eng', '') if bitrate.empty?
+      tags = stream.fetch('tags', {})
+      bitrate = stream.fetch('bit_rate', tags.fetch('BPS', tags.fetch('BPS-eng', '')))
       return nil if bitrate.empty?
 
       bitrate.to_i / 1000
@@ -1632,12 +1632,6 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       width   = video['width'].to_i
       height  = video['height'].to_i
 
-      if width == 720 and height == 576 and video['codec_name'] == 'mpeg2video'
-        pal = true
-      else
-        pal = false
-      end
-
       if crop.nil? or (crop == {:width => width, :height => height, :x => 0, :y => 0})
         crop_filter = nil
       else
@@ -1758,61 +1752,58 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         ]
       end
 
+      color_primaries = video['color_primaries']
+      color_trc       = video['color_transfer']
+      colorspace      = video['color_space']
       ten_bit = (@ten_bit ? (@eight_bit_vc1 ? (video['codec_name'] != 'vc1') : true) : false)
-      hdr = ((video.fetch('pix_fmt', 'yuv420p') == 'yuv420p10le') and ten_bit)
 
-      if hdr
-        color_primaries = 'bt2020'
-        color_trc       = 'smpte2084'
-        colorspace      = 'bt2020nc'
-      else
-        color_primaries = 'bt709'
-        color_trc       = 'bt709'
-        colorspace      = 'bt709'
+      if (video.fetch('pix_fmt', 'yuv420p') == 'yuv420p10le') and ten_bit
+        color_primaries ||= 'bt2020'
+        color_trc       ||= 'smpte2084'
+        colorspace      ||= 'bt2020nc'
       end
 
       if width > 1920 or height > 1080
+        level = '5.1'
         bitrate = @target_2160p
-        max_bitrate = 40000
-        max_dpb_mbs = 184320
+        max_bitrate = @hevc ? 25000 : 135000 # Level 5 HEVC Main 10 or H.264 Main
       elsif width > 1280 or height > 720
+        level = '4'
         bitrate = @target_1080p
-        max_bitrate = 20000
-        max_dpb_mbs = 32768
+        max_bitrate = @hevc ? 12000 : 20000 # Level 4
       elsif width > 720 or height > 576
+        level = '3.1'
         bitrate = @target_720p
-        max_bitrate = 10000
-        max_dpb_mbs = 18000
+        max_bitrate = @hevc ? 10000 : 14000 # Level 3.1
       else
+        color_primaries ||= (width == 720 and height == 576 and video['codec_name'] == 'mpeg2video') ? 'bt470bg' : 'smpte170m'
+        colorspace      ||= 'smpte170m'
+        level = '3'
         bitrate = @target_480p
-        max_bitrate = 5000
-        max_dpb_mbs = 8100
-
-        unless hdr
-          color_primaries = pal ? 'bt470bg' : 'smpte170m'
-          colorspace      = 'smpte170m'
-        end
+        max_bitrate = @hevc ? 6000 : 10000 # Level 3
       end
 
+      color_primaries ||= 'bt709'
+      color_trc       ||= 'bt709'
+      colorspace      ||= 'bt709'
       bitrate = @target unless @target.nil?
       bitrate = [bitrate, max_bitrate].min
-      maxrate = 0
-      bufsize = 0
 
       if @encoder =~ /(nvenc|hevc_qsv|libx26[45])$/
-        if @maxrate.nil?
-          maxrate = bitrate * 3
-        else
-          maxrate = [[(bitrate * @maxrate).to_i, (bitrate * 1.5).to_i].max, bitrate * 3].min
+        maxrate = [max_bitrate, bitrate * 2].max
+
+        unless @maxrate.nil?
+          maxrate = [[(@maxrate < bitrate ? bitrate * @maxrate : @maxrate).to_i, bitrate].max, maxrate].min
         end
 
-        if @bufsize.nil?
-          bufsize = maxrate if @encoder =~ /(nvenc|libx26[45])$/
-        else
-          unless @bufsize == 0 and @encoder =~ /nvenc$/
-            bufsize = [[(bitrate * @bufsize).to_i, bitrate].max, bitrate * 4].min
-          end
+        bufsize = @encoder == 'hevc_qsv' ? 0 : maxrate
+
+        unless @bufsize.nil?
+          bufsize = [[(@bufsize < bitrate ? bitrate * @bufsize : @bufsize).to_i, bitrate].max, maxrate].min
         end
+      else
+        maxrate = 0
+        bufsize = 0
       end
 
       unless @preset.nil?
@@ -1844,7 +1835,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
       text = "#{sprintf("%2d", video['index'])} = #{@encoder}"
 
       unless @encoder == 'copy'
-        text += " / #{bitrate} Kbps"
+        text += @nvenc_cq.nil? ? " / #{bitrate} Kbps" : " / #{@nvenc_cq} CQ"
         text += " / #{@preset}" unless @preset.nil?
       end
 
@@ -1854,21 +1845,25 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
 
       Kernel.warn text
       encode_options += ['-c:v', @encoder]
-      encode_options += ['-pix_fmt:v', (@encoder =~ /(nvenc|qsv)$/ ? 'p010le' : 'yuv420p10le')] if ten_bit
-      encode_options += ['-b:v', "#{bitrate}k"] unless @encoder == 'copy'
+      encode_options += ['-pix_fmt:v', (@encoder =~ /(videotoolbox|nvenc|qsv)$/ ? 'p010le' : 'yuv420p10le')] if ten_bit
+      encode_options += (@nvenc_cq.nil? ? ['-b:v', "#{bitrate}k"] : ['-cq:v', @nvenc_cq]) unless @encoder == 'copy'
       encode_options += ['-maxrate:v', "#{maxrate}k"] if maxrate > 0
       encode_options += ['-bufsize:v', "#{bufsize}k"] if bufsize > 0
       encode_options += ['-preset:v', @preset] unless @preset.nil?
-      encode_options += ['-allow_sw:v', '1'] if @encoder =~ /videotoolbox$/ and @vt_allow_sw
+
+      if @encoder =~ /videotoolbox$/
+        encode_options += ['-allow_sw:v', '1'] if @vt_allow_sw
+        encode_options += ['-profile:v', 'main10'] if ten_bit
+      end
 
       if @encoder =~ /nvenc$/
-        encode_options += ['-rc:v', @nvenc_rc_mode]
         encode_options += ['-spatial-aq:v', '1']                      if @nvenc_spatial_aq
         encode_options += ['-temporal-aq:v', '1']                     if @nvenc_temporal_aq
         encode_options += ['-rc-lookahead:v', @nvenc_lookahead.to_s]  unless @nvenc_lookahead.nil?
         encode_options += ['-multipass:v', @nvenc_multipass]          unless @nvenc_multipass.nil?
         encode_options += ['-refs:v', @nvenc_refs.to_s]               unless @nvenc_refs.nil?
         encode_options += ['-bf:v', @nvenc_bframes.to_s]              unless @nvenc_bframes.nil?
+        encode_options += ['-b_ref_mode:v', @nvenc_bframe_refs]       unless @nvenc_bframe_refs.nil?
       end
 
       if @encoder =~ /qsv$/
@@ -1904,20 +1899,10 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
             ]
           end
         else
-          max_refs = [(max_dpb_mbs / (((width + 15) / 16) * ((height + 15) / 16))), 16].min
-
           case @preset
-          when 'slow'
-            refs = 5
-          when 'slower'
-            refs = 8
-          when 'veryslow', 'placebo'
-            refs = 16
-          else
-            refs = 0
+          when 'slow', 'slower', 'veryslow', 'placebo'
+            encode_options += ['-level:v', level]
           end
-
-          encode_options += ['-refs:v', max_refs.to_s] if refs > max_refs
         end
 
         encode_options += ['-x264-params:v', @x264_params] unless @x264_params.nil?
@@ -1970,15 +1955,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
 
       audio_tracks = [{
         :stream => main_audio,
-        :width => width,
-        :bitrate => case width
-        when :stereo
-          @stereo_bitrate
-        when :surround
-          @surround_bitrate
-        when :original
-          nil
-        end
+        :width => width
       }]
 
       titles = {}
@@ -1992,15 +1969,6 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
 
         width = selection[:width]
 
-        bitrate = case width
-        when :stereo
-          @stereo_bitrate
-        when :surround
-          @surround_bitrate
-        when :original
-          nil
-        end
-
         unless selection[:track].nil?
           audio_track = 0
 
@@ -2012,8 +1980,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
             if audio_track == selection[:track]
               audio_tracks += [{
                 :stream => stream,
-                :width => width,
-                :bitrate => bitrate
+                :width => width
               }]
 
               break
@@ -2030,8 +1997,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
                 stream['index'] != main_audio['index']
               audio_tracks += [{
                 :stream => stream,
-                :width => width,
-                :bitrate => bitrate
+                :width => width
               }]
             end
           end
@@ -2046,8 +2012,7 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
             if title =~ /#{selection[:title]}/i and stream['index'] != main_audio['index']
               audio_tracks += [{
                 :stream => stream,
-                :width => width,
-                :bitrate => bitrate
+                :width => width
               }]
 
               titles[stream['index']] = title
@@ -2073,23 +2038,32 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         if track[:width] == :original
             encoder = 'copy'
         else
+          input_bitrate = track[:stream].fetch('bit_rate', '256').to_i / 1000
           dts = (codec_name == 'dts' and track[:stream].fetch('profile', 'DTS') =~ /^DTS(?:-ES)?$/)
 
           if track[:width] == :surround
-            if  ((codec_name == @surround_encoder or codec_name == 'ac3') and
-                (@keep_ac3_surround or (track[:stream]['bit_rate'].to_i / 1000) <= @surround_bitrate)) or
+            if  (@surround_encoder =~ /aac/ and codec_name == 'aac') or
+                (@surround_encoder =~ /ac3$/ and
+                (codec_name == @surround_encoder or codec_name == 'ac3') and
+                (@keep_ac3_surround or input_bitrate <= (@surround_bitrate.nil? ? 640 : @surround_bitrate))) or
                 (@pass_dts and dts)
               encoder = 'copy'
             elsif input_channels > 2
               encoder = @surround_encoder
               bitrate = @surround_bitrate
+
+              if encoder =~ /aac/
+                encoder = @aac_fallback_encoder if encoder == 'aac_at'
+                channels = 6
+              end
             end
           end
 
           if encoder.nil?
             if  input_channels <= 2 and (codec_name == 'aac' or
-                ((codec_name == @surround_encoder or codec_name == 'ac3') and
-                (@keep_ac3_stereo or (track[:stream]['bit_rate'].to_i / 1000) <= @stereo_bitrate)) or
+                (@surround_encoder =~ /ac3$/ and
+                (codec_name == @surround_encoder or codec_name == 'ac3') and
+                (@keep_ac3_stereo or input_bitrate <= (@stereo_bitrate.nil? ? 256 : @stereo_bitrate))) or
                 (@pass_dts and dts))
               encoder = 'copy'
             else
@@ -2118,19 +2092,19 @@ Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
         configurations[input_index] = configuration
         text = "#{sprintf("%2d", input_index)} = #{encoder}"
         text += " / #{bitrate} Kbps" unless bitrate.nil?
-        text += ' / stereo' unless channels.nil?
+        text += ' / stereo' unless channels.nil? or channels > 2
         text += " / #{titles[input_index]}" if titles.has_key?(input_index)
         Kernel.warn text
         copy_track_name = (@copy_track_names or titles.has_key?(input_index))
 
         options += [
           '-map', "0:#{input_index}",
-          "-c:a:#{index}", encoder
-        ] + (encoder == 'aac_at' ? ["-aac_at_mode:a:#{index}", 'cvbr'] : []) +
-            (bitrate.nil? ? [] : ["-b:a:#{index}", "#{bitrate}k"]) +
-            (channels.nil? ? [] : ["-ac:a:#{index}", "#{channels}"]) +
-            (track[:stream]['sample_rate'] != '48000' ? ["-ar:a:#{index}", '48000'] : []) +
-            (copy_track_name ? [] : ["-metadata:s:a:#{index}", 'title=']) + [
+          "-c:a:#{index}", encoder,
+          *(encoder == 'aac_at' ? ["-aac_at_mode:a:#{index}", 'cvbr'] : []),
+          *(bitrate.nil? ? [] : ["-b:a:#{index}", "#{bitrate}k"]),
+          *(channels.nil? ? [] : ["-ac:a:#{index}", "#{channels}"]),
+          *((encoder != 'copy' and track[:stream]['sample_rate'] != '48000') ? ["-ar:a:#{index}", '48000'] : []),
+          *(copy_track_name ? [] : ["-metadata:s:a:#{index}", 'title=']),
           "-disposition:a:#{index}", (index == 0 ? 'default' : '0')
         ]
 

data/other_video_transcoding.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name                  = 'other_video_transcoding'
-  s.version               = '0.8.0'
+  s.version               = '0.9.0'
   s.required_ruby_version = '>= 2.0'
   s.summary               = 'Other tools to transcode videos.'
   s.description           = <<-HERE

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: other_video_transcoding
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Don Melton
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-13 00:00:00.000000000 Z
+date: 2021-06-05 00:00:00.000000000 Z
 dependencies: []
 description: "    Other Video Transcoding is a package of tools to transcode videos.\n"
 email: don@blivet.com
@@ -35,6 +35,7 @@ files:
 - other_video_transcoding-0.5.0.gem
 - other_video_transcoding-0.6.0.gem
 - other_video_transcoding-0.7.0.gem
+- other_video_transcoding-0.8.0.gem
 - other_video_transcoding.gemspec
 homepage: https://github.com/donmelton/other_video_transcoding
 licenses: