other_video_transcoding 0.3.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cd29131688402599d3de0635a670005c8f32e7bda45f3b314dd7b47948d55be3
-  data.tar.gz: 1af9708f4b3639fd81d935b9c96ec120ecdc63411caa00a94783d44c2fb4d1f6
+  metadata.gz: e35c9d638ce45ae7407b8a3d88f0a785b515683c30e55b2d325e3cc243c15431
+  data.tar.gz: ae7a7405f0b57be44ee16dfc6b48c877730b4b6cd6933410a6ec3828bea124ef
 SHA512:
-  metadata.gz: 4ed431c314fc122ac855827c7ca831b67d1e7b5cb7dfe89500ce7bff6ec410a4e6893fc308f7724d8cb070dcc3c38eec92fbb57acd6e860690925c0691cde13a
-  data.tar.gz: 1442f616f7f8aed6aaafd2104f503c29e3bfa4c37a5f139c832f99b3a31c8f027f2636d7d2671734170d08dc97df53eae1b3d96270cef0badbf56d88ef95ca6c
+  metadata.gz: 15c29bac8aa3f7cb7ba9dcf6cf5c22e2c99f3b8ab8a709652166c885c5e2207873abdd2319c86cc5745aac977bab6911e50407356caa730976af465a88089dcb
+  data.tar.gz: 56d94453fda02a51ac29ddbdc904192874712f4ea3457a647383c2628bd54ebe636447771940e80ee6335a74e16ccef0fd14d6785e9a2998a22acf7b12c2f027

data/CHANGELOG.md

@@ -2,6 +2,57 @@
 
 This single document contains all of the notes created for each [release](https://github.com/donmelton/other_video_transcoding/releases).
 
+## [0.6.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.6.0)
+
+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. 
+* 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)
+
+Tuesday, November 24, 2020
+
+* Add `--qsv-decoder` and `--qsv-device` options to `other-transcode`, both of which enable the scoped use of the Intel Quick Sync Video (QSV) decoder instead of the generic hardware decoder. These options can significantly speed operation of the QSV encoder, invoked via `--qsv`. It's recommended that `--decode all` be included when using these options to decode all video input formats. The `--qsv-device` option allows selection of specific hardware by number or path depending on platform. Please note that deinterlacing, cropping, scaling or using other filters will disable QSV's format-specific decoders.
+* Remove all deprecated options and arguments from `other-transcode`.
+
+## [0.4.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.4.0)
+
+Sunday, November 1, 2020
+
+* Modify the behavior and augment the capabilities of both encoding and decoding when using Nvidia hardware with `other-transcode`. This is necessary for compatibility with unleased versions of `ffmpeg` which are now built using the new Nvidia Software Development Kit (SDK) version 11.0. This SDK changes default encoder behavior and adds new presets which allow finer control of the performance/quality trade-off when transcoding video. To allow maximum performance, `other-transcode` no longer enables some quality settings by default:
+    * Multipass mode, now accessible via a new `--nvenc-multipass` option. Be advised that any improved quality from enabling multipass mode is probably not worth the performance impact.
+    * Spatial and temporal adaptive quantization (AQ), accessible via the `--nvenc-spatial-aq` and `--nvenc-temporal-aq` options. While enabling spatial AQ is still useful in reducing color banding for some inputs, be advised that enabling temporal AQ is probably not necessary and can cause some other side effects.
+* Add support for seven new Nvidia encoder presets to `other-transcode`. Use `--preset p1` for best performance and `--preset p7` for best quality. It's not necessary to use `--preset p4` since that's the default. See the [Nvidia preset migration guide](https://docs.nvidia.com/video-technologies/video-codec-sdk/nvenc-preset-migration-guide/index.html) to understand how these presets work and how they map to older behavior.
+* Add a `--nvenc-rc-mode` option to `other-transcode` for backward comaptibility with `ffmpeg` version 4.3.1 and older.
+* Add `--cuda` and `--no-cuda` options to `other-transcode`. These options enable or disable the scoped use of the Nvidia CUDA hardware decoder instead of the generic hardware decoder. By default the CUDA _decoder_ is enabled when using the Nvidia video _encoder_, but disabled when using other encoders.
+* Deprecate the `--cuvid` option in `other-transcode` because the CUDA decoder is faster and more flexible.
+* Deprecate `--preset none` in `other-transcode` because it's no longer necessary.
+* Always use hyphen-based spellings of Nvidia AQ options in `ffmpeg` commands generated by `other-transcode`.
+* Add `--x264-params` and `--x265-params` options to `other-transcode` for _very_ advanced manipulation of the `x264` and `x265` software encoders.
+* Modify `other-transcode` to assume a video input without a `field_order` tag is progressive instead of interlaced so a deinterlace fliter is not automatically and incorrectly applied to that video. This avoids problems with some 4K Ultra HD Blu-ray rips.
+* Update the link to Docker containers for Linux in the "README" document. Thanks, @ttyS0!
+
+## [0.3.2](https://github.com/donmelton/other_video_transcoding/releases/tag/0.3.2)
+
+Friday, September 11, 2020
+
+* Modify `other-transcode` to use a new `ffmpeg` Matroksa muxer option so the `-disposition` option is once again honored when using `ffmpeg` version 4.3 and later.
+* Change the codec ID from the default of `hev1` to `hvc1` for HEVC video in MP4 output from `other-transcode` to enable playback in QuickTime on macOS. Via [ #50](https://github.com/donmelton/other_video_transcoding/issues/50).
+* Convert added SRT format subtitles to MOV-compatible format in MP4 output from `other-transcode`. Via [ #55](https://github.com/donmelton/other_video_transcoding/issues/55).
+
+## [0.3.1](https://github.com/donmelton/other_video_transcoding/releases/tag/0.3.1)
+
+Tuesday, May 26, 2020
+
+* Modify the `--preview-crop` option in `other-transcode` to show commands compatible with newer versions of `mpv`.
+* No longer force a NTSC film frame rate for interlaced inputs in PAL MPEG-2 format.
+* When using the `--dry-run` option in `other-transcode`, issue a warning instead of failing if the output or log files already exist.
+* Add a link to another Docker container for Linux in the "README" document. Thanks, @ttyS0!
+
 ## [0.3.0](https://github.com/donmelton/other_video_transcoding/releases/tag/0.3.0)
 
 Thursday, February 27, 2020

data/README.md

@@ -62,9 +62,9 @@ See "[Download FFmpeg](https://ffmpeg.org/download.html)," "[MKVToolNix Download
 
 Additional documentation for installing these programs on Windows is available in the [wiki](https://github.com/donmelton/other_video_transcoding/wiki/Windows).
 
-A [Docker](https://en.wikipedia.org/wiki/Docker_(software)) container for Linux, including installation instructions, is available here:
+[Docker](https://en.wikipedia.org/wiki/Docker_(software)) containers for Linux, including installation instructions, are available here:
 
-https://github.com/ttyS0/docker-transcode-vaapi
+https://github.com/ttyS0/docker-other-transcode
 
 On macOS, all of these programs can be easily installed via [Homebrew](http://brew.sh/), an optional package manager:
 

data/bin/ask-ffmpeg-log

@@ -15,14 +15,14 @@ module Transcoding
 
   class Command
     def about
-      <<HERE
-ask-ffmpeg-log 0.3.0
+      <<-HERE
+ask-ffmpeg-log 0.6.0
 Copyright (c) 2019-2020 Don Melton
-HERE
+      HERE
     end
 
     def usage
-      <<HERE
+      <<-HERE
 Report temporal information from ffmpeg-generated `.log` files
 containing encoding statistics.
 
@@ -34,7 +34,7 @@ Options:
     --tabular       use tab character as field delimiter and suppress labels
 -h, --help          display this help and exit
     --version       output version information and exit
-HERE
+      HERE
     end
 
     def initialize

data/bin/other-transcode

@@ -17,14 +17,14 @@ module Transcoding
 
   class Command
     def about
-      <<HERE
-other-transcode 0.3.0
+      <<-HERE
+other-transcode 0.6.0
 Copyright (c) 2019-2020 Don Melton
-HERE
+      HERE
     end
 
     def usage1
-      <<HERE
+      <<-HERE
 Transcode Blu-ray Disc or DVD rip into a smaller, more portable format
 while remaining high enough quality to be mistaken for the original.
 
@@ -34,55 +34,55 @@ Creates Matroska `.mkv` format file in current working directory.
 
 Automatically selects a platform-specific hardware video encoder.
 
-HERE
+      HERE
     end
 
     def usage2
-      <<HERE
+      <<-HERE
 Input options:
     --position TIME, --duration TIME
                     start transcoding at position and/or limit to duration
                       in seconds[.milliseconds] or [HH:]MM:SS[.m...] format
 
-HERE
+      HERE
     end
 
     def usage3
-      <<HERE
+      <<-HERE
 Output options:
     --debug         increase diagnostic information
     --scan          print media information and exit
     --preview-crop  show commands to preview detected video crop and exit
-HERE
+      HERE
     end
 
     def usage4
-      <<HERE
+      <<-HERE
     --print-crop    print only detected video crop geometry and exit
     --mp4           output MP4 instead of Matroska `.mkv` format
     --copy-track-names
                     copy all input audio track names to output
-HERE
+      HERE
     end
 
     def usage5
-      <<HERE
+      <<-HERE
     --max-muxing-queue-size SIZE
                     set maximum number of packets to buffer when muxing
-HERE
+      HERE
     end
 
     def usage6
-      <<HERE
+      <<-HERE
 -n, --dry-run       don't transcode, just show `ffmpeg` command and exit
 
 Video options:
     --hevc          use HEVC version of platform-specific video encoder
-HERE
+      HERE
     end
 
     def usage7
-      <<HERE
+      <<-HERE
     --vt            use Apple Video Toolbox encoder
     --nvenc         use Nvidia video encoder
     --qsv           use Intel Quick Sync video encoder
@@ -93,19 +93,24 @@ HERE
     --10-bit, --no-10-bit
                     use 10-bit pixel format (default: not used for H.264,
                       used for HEVC with Nvidia, Intel and x265 encoders)
-    --preset NAME|none
-                    apply video encoder preset or disable default settings
+    --preset NAME   apply video encoder preset
     --decode vc1|all|none
                     set scope of automatic hardware decoder acceleration
                       (default: vc1 for VC-1 format only)
-    --cuvid         use Nvidia video decoder
-                      for H.264, VC-1, MPEG-2 and other formats
-                      (ignores scope set by `--decode`)
-HERE
+    --cuda, --no-cuda
+                    enable or disable scoped use of Nvidia video decoder
+                      instead of generic hardware decoder
+                      (default: enabled when using Nvidia video encoder,
+                                disabled when using other encoders)
+    --qsv-decoder   enable scoped use of Intel Quick Sync video decoder
+                      instead of generic hardware decoder
+    --qsv-device DEVICE
+                    enable scoped use of QSV decoder for specific device
+      HERE
     end
 
     def usage8
-      <<HERE
+      <<-HERE
     --target [2160p=|1080p=|720p=|480p=]BITRATE
                     set video bitrate target (default: based on input)
                       or target for specific input resolution
@@ -113,11 +118,11 @@ HERE
                     set video crop geometry (default: none)
                       or automatically detect it
     --720p          fit video within 1280x720 pixel bounds
-HERE
+      HERE
     end
 
     def usage9
-      <<HERE
+      <<-HERE
     --1080p          "    "     "    1920x1080  "     "
     --deinterlace   reduce interlace artifacts without changing frame rate
                       (applied automatically for some inputs)
@@ -126,11 +131,11 @@ HERE
     --detelecine    drop duplicate frames to restore original frame rate
                       (disables any deinterlacing and forced frame rate)
     --no-filters    disable any automatic adjustments via filters
-HERE
+      HERE
     end
 
     def usage10
-      <<HERE
+      <<-HERE
     --rc-maxrate FACTOR, --rc-bufsize FACTOR
                     set ratecontrol maximum rate and/or buffer size
                       as multiple of video bitrate target
@@ -140,17 +145,20 @@ Apple Video Toolbox encoder options:
     --vt-allow-sw   allow software encoding
 
 Nvidia video encoder options:
-    --nvenc-spatial-aq, --no-nvenc-spatial-aq
-                    enable or disable spatial AQ (default: enabled)
-    --nvenc-temporal-aq, --no-nvenc-temporal-aq
-                    enable or disable temporal AQ
-                      (default: enabled for H.264, disabled for HEVC)
+    --nvenc-spatial-aq
+                    enable spatial adaptive quantization (AQ)
+    --nvenc-temporal-aq
+                    enable temporal adaptive quantization (AQ)
     --nvenc-lookahead FRAMES
                     set number of frames to look ahead for ratecontrol
+    --nvenc-multipass qres|fullres
+                    set multipass encoding resolution
     --nvenc-refs FRAMES
                     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)
 
 Intel Quick Sync video encoder options:
     --qsv-refs FRAMES
@@ -179,27 +187,33 @@ x264 software video encoder options:
     --x264-quick    increase encoding speed by 70-80%
                       with no easily perceptible loss in video quality
                       (avoids quality problems with some encoder presets)
-HERE
+    --x264-params KEY=VALUE[:KEY=VALUE]...
+                    override x264 configuration (disables other x264 options)
+
+x265 software video encoder options:
+    --x265-params KEY=VALUE[:KEY=VALUE]...
+                    override x265 configuration
+      HERE
     end
 
     def usage11
-      <<HERE
+      <<-HERE
 
 Audio options:
     --main-audio TRACK[=WIDTH]
                     select main audio track by number (default: 1)
                       with optional width (default: surround)
-HERE
+      HERE
     end
 
     def usage12
-      <<HERE
+      <<-HERE
                         (use `original` to disable transcoding)
-HERE
+      HERE
     end
 
     def usage13
-      <<HERE
+      <<-HERE
     --add-audio TRACK|all|LANGUAGE|STRING[=WIDTH]
                     add single audio track by number
                       including main audio track
@@ -212,49 +226,52 @@ HERE
                       excluding main audio track
                       (comparison is case-insensitve)
                     with optional width (default: stereo)
-HERE
+      HERE
     end
 
     def usage14
-      <<HERE
+      <<-HERE
                       (use `original` to disable transcoding)
-HERE
+      HERE
     end
 
     def usage15
-      <<HERE
+      <<-HERE
     --surround-bitrate BITRATE
                     set surround audio bitrate (default: 640)
     --stereo-bitrate BITRATE
                     set stereo audio bitrate (default: 256)
-HERE
+      HERE
     end
 
     def usage16
-      <<HERE
+      <<-HERE
     --mono-bitrate BITRATE
                     set mono audio bitrate (default: 50% of stereo bitrate)
-HERE
+      HERE
     end
 
     def usage17
-      <<HERE
+      <<-HERE
     --eac3          use Enhanced AC-3 format for surround audio
-HERE
+      HERE
     end
 
     def usage18
-      <<HERE
+      <<-HERE
     --all-eac3       "     "      "     "     "  all audio
+    --limit-ac3-surround
+                    don't copy surround audio in AC-3 format
+                      when orginal bitrate is above transcoding bitrate
     --keep-ac3-stereo
                     copy stereo and mono audio in AC-3 format
                       even when orginal bitrate is above transcoding bitrate
     --pass-dts      enable passthrough of audio in DTS and DTS-ES formats
-HERE
+      HERE
     end
 
     def usage19
-      <<HERE
+      <<-HERE
 
 Subtitle options:
     --add-subtitle TRACK[=forced]|auto|all|LANGUAGE|STRING
@@ -279,7 +296,7 @@ Other options:
     --version       output version information and exit
 
 Requires `ffprobe`, `ffmpeg` and `mkvpropedit`.
-HERE
+      HERE
     end
 
     def initialize
@@ -290,6 +307,7 @@ HERE
       @detect = false
       @preview = false
       @format = :mkv
+      @mkv_options = []
       @copy_track_names = false
       @max_muxing_queue_size = nil
       @dry_run = false
@@ -298,7 +316,8 @@ HERE
       @ten_bit = nil
       @preset = nil
       @decode_scope = :vc1
-      @decoder_type = nil
+      @decode_method = nil
+      @qsv_device = nil
       @target_2160p = nil
       @target_1080p = nil
       @target_720p  = nil
@@ -314,11 +333,13 @@ HERE
       @maxrate = nil
       @bufsize = nil
       @vt_allow_sw = false
-      @nvenc_spatial_aq = nil
-      @nvenc_temporal_aq = nil
+      @nvenc_spatial_aq = false
+      @nvenc_temporal_aq = false
       @nvenc_lookahead = nil
+      @nvenc_multipass = nil
       @nvenc_refs = nil
       @nvenc_bframes = nil
+      @nvenc_rc_mode = 'vbr'
       @qsv_refs = nil
       @qsv_bframes = nil
       @amf_quality = nil
@@ -330,6 +351,8 @@ HERE
       @x264_avbr = false
       @x264_mbtree = false
       @x264_quick = false
+      @x264_params = nil
+      @x265_params = nil
       @audio_selections = [{
         :track => 1,
         :language => nil,
@@ -341,6 +364,7 @@ HERE
       @mono_bitrate = nil
       @surround_encoder = 'ac3'
       @stereo_encoder = nil
+      @keep_ac3_surround = true
       @keep_ac3_stereo = false
       @pass_dts = false
       @subtitle_selections = []
@@ -495,8 +519,17 @@ HERE
         end
       end
 
-      opts.on '--cuvid' do
-        @decoder_type = :cuvid
+      opts.on '--[no-]cuda' do |arg|
+        @decode_method = arg ? 'cuda' : 'auto'
+      end
+
+      opts.on '--qsv-decoder' do
+        @decode_method = 'qsv'
+      end
+
+      opts.on '--qsv-device ARG' do |arg|
+        @qsv_device = arg
+        @decode_method = 'qsv'
       end
 
       opts.on '--target ARG' do |arg|
@@ -553,7 +586,7 @@ HERE
         @deinterlace = true
         @detelecine = false
         @enable_filters = false
-        @encoder = nil if @encoder == 'copy' and @decoder_type != :cuvid
+        @encoder = nil if @encoder == 'copy'
       end
 
       opts.on '--rate ARG' do |arg|
@@ -619,19 +652,35 @@ HERE
         @vt_allow_sw = true
       end
 
-      opts.on '--[no-]nvenc-spatial-aq' do |arg|
+      opts.on '--nvenc-spatial-aq' do
         @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
-        @nvenc_spatial_aq = arg
+        @nvenc_spatial_aq = true
       end
 
-      opts.on '--[no-]nvenc-temporal-aq' do |arg|
+      opts.on '--nvenc-temporal-aq' do
         @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
-        @nvenc_temporal_aq = arg
+        @nvenc_temporal_aq = true
       end
 
       opts.on '--nvenc-lookahead ARG', Integer do |arg|
         @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
-        @nvenc_lookahead = [[arg, 0].max, 32].min
+
+        if arg > 0
+          @nvenc_lookahead = [arg, 32].min
+        else
+          @nvenc_lookahead = nil
+        end
+      end
+
+      opts.on '--nvenc-multipass ARG' do |arg|
+        @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
+
+        @nvenc_multipass = case arg
+        when 'qres', 'fullres'
+          arg
+        else
+          fail UsageError, "invalid multipass resolution argument: #{arg}"
+        end
       end
 
       opts.on '--nvenc-refs ARG', Integer do |arg|
@@ -644,6 +693,17 @@ HERE
         @nvenc_bframes = [[arg, 0].max, 4].min
       end
 
+      opts.on '--nvenc-rc-mode ARG' do |arg|
+        @encoder = @hevc ? 'hevc_nvenc' : 'h264_nvenc'
+
+        @nvenc_rc_mode = case arg
+        when 'vbr', 'vbr_hq'
+          arg
+        else
+          fail UsageError, "invalid rate control argument: #{arg}"
+        end
+      end
+
       opts.on '--qsv-refs ARG', Integer do |arg|
         @encoder = @hevc ? 'hevc_qsv' : 'h264_qsv'
         @qsv_refs = [arg, 0].max
@@ -695,6 +755,7 @@ HERE
         @hevc = false
         @x264_avbr = true
         @x264_mbtree = false
+        @x264_params = nil
       end
 
       opts.on '--x264-mbtree' do
@@ -702,15 +763,40 @@ HERE
         @hevc = false
         @x264_mbtree = true
         @x264_avbr = false
+        @x264_params = nil
       end
 
       opts.on '--x264-quick' do
         @encoder = 'libx264'
         @hevc = false
         @x264_quick = true
+        @x264_params = nil
         @preset = nil
       end
 
+      opts.on '--x264-params ARG' do |arg|
+        arg.split ':' do |param|
+          fail UsageError, "invalid argument: #{arg}" unless param =~ /^[\w\-]+=[\w\-\.,]+$/
+        end
+
+        @encoder = 'libx264'
+        @hevc = false
+        @x264_params = arg
+        @x264_avbr = false
+        @x264_mbtree = true
+        @x264_quick = false
+      end
+
+      opts.on '--x265-params ARG' do |arg|
+        arg.split ':' do |param|
+          fail UsageError, "invalid argument: #{arg}" unless param =~ /^[\w\-]+=[\w\-\.,]+$/
+        end
+
+        @encoder = 'libx265'
+        @hevc = true
+        @x265_params = arg
+      end
+
       opts.on '--main-audio ARG' do |arg|
         if arg =~ /^([0-9]+)(?:=(stereo|surround|original))?$/
           @audio_selections[0][:track] = $1.to_i
@@ -769,6 +855,10 @@ HERE
         @stereo_encoder   = 'eac3'
       end
 
+      opts.on '--limit-ac3-surround' do
+        @keep_ac3_surround = false
+      end
+
       opts.on '--keep-ac3-stereo' do
         @keep_ac3_stereo = true
       end
@@ -827,13 +917,14 @@ HERE
     def configure(path)
       @audio_selections.uniq!
       @subtitle_selections.uniq!
-      @surround_bitrate = [[@surround_bitrate, 256].max, (@surround_encoder == 'ac3' ? 640 : 768)].min
-      @stereo_bitrate = [[@stereo_bitrate, 128].max, (@stereo_encoder == 'eac3' ? 768 : 320)].min
+      min_bitrate = @surround_encoder == 'eac3' ? 192 : 256
+      @surround_bitrate = [[@surround_bitrate, min_bitrate].max, (@surround_encoder == 'ac3' ? 640 : 768)].min
+      @stereo_bitrate = [[@stereo_bitrate, min_bitrate / 2].max, (@stereo_encoder == 'eac3' ? 768 : 320)].min
 
       if @mono_bitrate.nil?
         @mono_bitrate = @stereo_bitrate / 2
       else
-        @mono_bitrate = [[@mono_bitrate, 64].max, (@stereo_encoder == 'eac3' ? 768 : 256)].min
+        @mono_bitrate = [[@mono_bitrate, min_bitrate / 4].max, (@stereo_encoder == 'eac3' ? 768 : 256)].min
       end
 
       [
@@ -875,10 +966,11 @@ HERE
       end
 
       @ten_bit = (@hevc and @encoder =~ /(nvenc|qsv|x265)$/ ? true : false) if @ten_bit.nil?
-      @target_2160p ||= (@hevc and @ten_bit) ? 12000 : 16000
-      @target_1080p ||= (@hevc and @ten_bit) ? 6000  : 8000
-      @target_720p  ||= (@hevc and @ten_bit) ? 3000  : 4000
-      @target_480p  ||= (@hevc and @ten_bit) ? 1500  : 2000
+      @target_2160p ||= @hevc ? 12000 : 16000
+      @target_1080p ||= @hevc ? 6000  : 8000
+      @target_720p  ||= @hevc ? 3000  : 4000
+      @target_480p  ||= @hevc ? 1500  : 2000
+      @decode_method ||= @encoder =~ /nvenc$/ ? 'cuda' : 'auto'
 
       if @stereo_encoder.nil?
         if encoders =~ /aac_at/ or encoders =~ /libfdk_aac/
@@ -887,6 +979,11 @@ HERE
           @stereo_encoder = 'aac'
         end
       end
+
+      if @format == :mkv
+        capabilities = get_muxer_capabilities
+        @mkv_options = ['-default_mode', 'passthrough'] if capabilities =~ /passthrough/
+      end
     end
 
     def verify_tool_availability(command)
@@ -942,8 +1039,7 @@ HERE
         ] + (encoder =~ /vaapi$/ ? ['-filter:v', 'format=nv12,hwupload'] : []) + [
           '-c:v', encoder,
           '-b:v', '1000k'
-        ] + (encoder =~ /nvenc$/ ? ['-rc:v', 'vbr_hq', '-spatial-aq:v', '1'] : []) +
-            (encoder == 'h264_nvenc' ? ['-temporal-aq:v', '1'] : []) +
+        ] + (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'] : []) + [
@@ -964,18 +1060,42 @@ HERE
       $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
 
       unless @scan or @detect
         output_path = File.basename(path, '.*') + '.' + @format.to_s
-        fail "output file already exists: #{output_path}" if File.exist? output_path
+        fail_or_warn "output file already exists: #{output_path}" if File.exist? output_path
 
         log_path = output_path + '.log'
-        fail "log file already exists: #{log_path}" if File.exist? log_path
+        fail_or_warn "log file already exists: #{log_path}" if File.exist? log_path
 
         tmp_log_path = "_ffmpeg_#{rand(10000..99999)}_#{$PROCESS_ID}.#{@format.to_s}.log"
-        fail "log file already exists: #{tmp_log_path}" if File.exist? tmp_log_path
+        fail_or_warn "log file already exists: #{tmp_log_path}" if File.exist? tmp_log_path
       end
 
       media_info = scan_media(path)
@@ -1034,7 +1154,7 @@ HERE
           get_audio_options(media_info) +
           get_subtitle_options(media_info, burn_subtitle) + [
         '-metadata:g', 'title='
-      ] + (@format == :mp4 ? ['-movflags', 'disable_chpl'] : []) + [
+      ] + (@format == :mkv ? @mkv_options : ['-movflags', 'disable_chpl']) + [
         output_path
       ]
 
@@ -1090,6 +1210,14 @@ HERE
       Kernel.warn "\nElapsed time: #{seconds_to_time(Time.now.tv_sec - seconds)}\n\n"
     end
 
+    def fail_or_warn(message)
+      if @dry_run
+        Kernel.warn "#{$PROGRAM_NAME}: #{message}"
+      else
+        fail message
+      end
+    end
+
     def scan_media(path)
       Kernel.warn 'Scanning media...'
       output = ''
@@ -1333,10 +1461,10 @@ HERE
         drawbox_string = "#{crop[:x]}:#{crop[:y]}:#{crop[:width]}:#{crop[:height]}"
         puts
         puts escape_command([
-          'mpv', '--no-audio', '--vf', "lavfi=[drawbox=#{drawbox_string}:invert:1]", path
+          'mpv', '--no-audio', "--vf=lavfi=[drawbox=#{drawbox_string}:invert:1]", path
         ])
         puts escape_command([
-          'mpv', '--no-audio', '--vf', "crop=#{crop_string}", path
+          'mpv', '--no-audio', "--vf=crop=#{crop_string}", path
         ])
         puts
         puts escape_command([
@@ -1435,48 +1563,20 @@ HERE
     end
 
     def get_video_options(media_info, video, burn_subtitle, crop)
-      if @decoder_type == :cuvid
-        cuvid_decoder = case video['codec_name']
-        when 'mpeg1video'
-          'mpeg1_cuvid'
-        when 'mpeg2video'
-          'mpeg2_cuvid'
-        when 'mjpeg'
-          'mjpeg_cuvid'
-        when 'mpeg4'
-          'mpeg4_cuvid'
-        when 'h264'
-          'h264_cuvid'
-        when 'vc1'
-          'vc1_cuvid'
-        when 'vp8'
-          'vp8_cuvid'
-        when 'vp9'
-          'vp9_cuvid'
-        when 'hevc'
-          'hevc_cuvid'
-        end
-      else
-        cuvid_decoder = nil
-      end
-
-      cuvid_options = []
-
       if burn_subtitle.nil?
         overlay_filter = nil
       else
         overlay_filter = "[0:#{burn_subtitle['index']}]overlay"
-        cuvid_decoder = nil
       end
 
       deinterlace = @deinterlace
       rate = @rate
 
       if @enable_filters
-        if video['avg_frame_rate'] == '30000/1001' or video['field_order'] != 'progressive'
+        if video['avg_frame_rate'] == '30000/1001' or video.fetch('field_order', 'progressive') != 'progressive'
           deinterlace = true
 
-          if video['codec_name'] == 'mpeg2video'
+          if video['codec_name'] == 'mpeg2video' and video['avg_frame_rate'] != '25/1'
             rate = '24000/1001'
           end
         end
@@ -1485,11 +1585,7 @@ HERE
       frame_rate_filter = nil
 
       if deinterlace
-        if cuvid_decoder.nil?
-          frame_rate_filter = 'yadif=deint=interlaced' unless @encoder == 'copy'
-        else
-          cuvid_options += ['-deint:v', 'adaptive']
-        end
+        frame_rate_filter = 'yadif=deint=interlaced' unless @encoder == 'copy'
       end
 
       unless rate.nil?
@@ -1500,7 +1596,6 @@ HERE
 
       if @detelecine
         frame_rate_filter = 'fieldmatch=order=tff:combmatch=none,decimate'
-        cuvid_decoder = nil
       end
 
       width   = video['width'].to_i
@@ -1515,21 +1610,9 @@ HERE
       if crop.nil? or (crop == {:width => width, :height => height, :x => 0, :y => 0})
         crop_filter = nil
       else
-        media_width   = width
-        media_height  = height
-        width         = crop[:width]
-        height        = crop[:height]
-
-        if cuvid_decoder.nil?
-          crop_filter = "crop=#{width}:#{height}:#{crop[:x]}:#{crop[:y]}"
-        else
-          crop_filter = nil
-          top     = crop[:y]
-          bottom  = media_height - (top + height)
-          left    = crop[:x]
-          right   = media_width - (left + width)
-          cuvid_options += ['-crop:v', "#{top}x#{bottom}x#{left}x#{right}"]
-        end
+        width       = crop[:width]
+        height      = crop[:height]
+        crop_filter = "crop=#{width}:#{height}:#{crop[:x]}:#{crop[:y]}"
       end
 
       if @hevc
@@ -1544,14 +1627,8 @@ HERE
         scale = [(max_width.to_f / width), (max_height.to_f / height)].min
         width   = ((width   * scale).ceil / 2) * 2
         height  = ((height  * scale).ceil / 2) * 2
-
-        if cuvid_decoder.nil?
-          scale_filter = "scale=#{width}:#{height}"
-          scale_filter += ':flags=bicubic' unless overlay_filter.nil?
-        else
-          scale_filter = nil
-          cuvid_options += ['-resize:v', "#{width}x#{height}"]
-        end
+        scale_filter = "scale=#{width}:#{height}"
+        scale_filter += ':flags=bicubic' unless overlay_filter.nil?
       else
         scale_filter = nil
       end
@@ -1562,24 +1639,52 @@ HERE
         decode_options = []
       end
 
-      if cuvid_decoder.nil?
-        if (@decode_scope == :vc1 and video['codec_name'] == 'vc1') or @decode_scope == :all
-          if @encoder =~ /vaapi$/
-            decode_options = [
-              '-hwaccel', 'vaapi',
-              '-hwaccel_device', '/dev/dri/renderD128',
-              '-hwaccel_output_format', 'vaapi'
-            ]
+      if (@decode_scope == :vc1 and video['codec_name'] == 'vc1') or @decode_scope == :all
+        if @encoder =~ /vaapi$/
+          decode_options = [
+            '-hwaccel', 'vaapi',
+            '-hwaccel_device', '/dev/dri/renderD128',
+            '-hwaccel_output_format', 'vaapi'
+          ]
+        else
+          if @decode_method == 'qsv' and @encoder != 'h264_qsv'
+            decode_method = 'auto'
           else
-            decode_options += ['-hwaccel', 'auto']
+            decode_method = @decode_method
           end
-        end
-      else
-        Kernel.warn "video decoder = #{cuvid_decoder}"
 
-        decode_options += [
-          '-c:v', cuvid_decoder
-        ] + cuvid_options
+          decode_options += ['-hwaccel', decode_method]
+
+          if  decode_method == 'qsv' and
+              overlay_filter.nil? and
+              frame_rate_filter.nil? and
+              crop_filter.nil? and
+              scale_filter.nil?
+            qsv_decoder = case video['codec_name']
+            when 'av1'
+              'av1_qsv'
+            when 'h264'
+              'h264_qsv'
+            when 'hevc'
+              'hevc_qsv'
+            when 'mjpeg'
+              'mjpeg_qsv'
+            when 'mpeg2video'
+              'mpeg2_qsv'
+            when 'vc1'
+              'vc1_qsv'
+            when 'vp8'
+              'vp8_qsv'
+            when 'vp9'
+              'vp9_qsv'
+            else
+              nil
+            end
+
+            decode_options += ['-qsv_device', @qsv_device] unless @qsv_device.nil?
+            decode_options += ['-c:v', qsv_decoder] unless qsv_decoder.nil?
+          end
+        end
       end
 
       if @encoder =~ /vaapi$/ and not decode_options.include?('-hwaccel')
@@ -1664,21 +1769,21 @@ HERE
         end
 
         if @bufsize.nil?
-          bufsize = maxrate if @encoder =~ /^libx26[45]$/
+          bufsize = maxrate if @encoder =~ /(nvenc|libx26[45])$/
         else
-          bufsize = [[(bitrate * @bufsize).to_i, bitrate].max, bitrate * 4].min
+          unless @bufsize == 0 and @encoder =~ /nvenc$/
+            bufsize = [[(bitrate * @bufsize).to_i, bitrate].max, bitrate * 4].min
+          end
         end
       end
 
-      if @preset.nil? or @preset == 'none'
-        preset = nil
-      else
+      unless @preset.nil?
         valid = false
 
         case @encoder
         when /nvenc$/
           case @preset
-          when 'fast', 'medium', 'slow'
+          when 'fast', 'medium', 'slow', /^p[1-7]$/
             valid = true
           end
         when /qsv$/
@@ -1695,8 +1800,6 @@ HERE
         end
 
         fail "invalid preset for encoder: #{@preset}" unless valid
-
-        preset = @preset
       end
 
       Kernel.warn 'Stream mapping:'
@@ -1704,7 +1807,7 @@ HERE
 
       unless @encoder == 'copy'
         text += " / #{bitrate} Kbps"
-        text += " / #{preset}" unless preset.nil?
+        text += " / #{@preset}" unless @preset.nil?
       end
 
       unless burn_subtitle.nil?
@@ -1717,33 +1820,15 @@ HERE
       encode_options += ['-b:v', "#{bitrate}k"] 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 += ['-preset:v', @preset] unless @preset.nil?
       encode_options += ['-allow_sw:v', '1'] if @encoder =~ /videotoolbox$/ and @vt_allow_sw
 
       if @encoder =~ /nvenc$/
-        spatial_aq  = @nvenc_spatial_aq.nil?  ? false : @nvenc_spatial_aq
-        temporal_aq = @nvenc_temporal_aq.nil? ? false : @nvenc_temporal_aq
-
-        if @hevc
-          spatial_aq_option   = '-spatial_aq:v'
-          temporal_aq_option  = '-temporal_aq:v'
-        else
-          spatial_aq_option   = '-spatial-aq:v'
-          temporal_aq_option  = '-temporal-aq:v'
-        end
-
-        if @preset.nil?
-          encode_options += ['-rc:v', 'vbr_hq']
-          spatial_aq = true if @nvenc_spatial_aq.nil?
-
-          unless @hevc
-            temporal_aq = true if @nvenc_temporal_aq.nil?
-          end
-        end
-
-        encode_options += [spatial_aq_option, '1']                    if spatial_aq
-        encode_options += [temporal_aq_option, '1']                   if temporal_aq
+        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?
       end
@@ -1796,6 +1881,12 @@ HERE
 
           encode_options += ['-refs:v', max_refs.to_s] if refs > max_refs
         end
+
+        encode_options += ['-x264-params:v', @x264_params] unless @x264_params.nil?
+      end
+
+      if @encoder == 'libx265'
+        encode_options += ['-x265-params:v', @x265_params] unless @x265_params.nil?
       end
 
       unless @ten_bit
@@ -1815,6 +1906,8 @@ HERE
         '-disposition:v', 'default'
       ]
 
+      encode_options += ['-tag:v', 'hvc1'] if @format == :mp4 and @hevc
+
       [decode_options, encode_options]
     end
 
@@ -1945,8 +2038,8 @@ HERE
           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' or
+            if  ((codec_name == @surround_encoder or codec_name == 'ac3') and
+                (@keep_ac3_surround or (track[:stream]['bit_rate'].to_i / 1000) <= @surround_bitrate)) or
                 (@pass_dts and dts)
               encoder = 'copy'
             elsif input_channels > 2
@@ -2087,7 +2180,7 @@ HERE
 
         options += [
           '-map', "0:#{subtitle['index']}",
-          "-c:s:#{index}", 'copy',
+          "-c:s:#{index}", ((@format == :mp4 and subtitle['codec_name'] == 'subrip') ? 'mov_text' : 'copy'),
           "-disposition:s:#{index}", (force ? 'default+forced' : '0')
         ]
 

data/other_video_transcoding.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name                  = 'other_video_transcoding'
-  s.version               = '0.3.0'
+  s.version               = '0.6.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.3.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Don Melton
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-02-28 00:00:00.000000000 Z
+date: 2020-12-23 00:00:00.000000000 Z
 dependencies: []
 description: "    Other Video Transcoding is a package of tools to transcode videos.\n"
 email: don@blivet.com
@@ -25,12 +25,20 @@ files:
 - README.md
 - bin/ask-ffmpeg-log
 - bin/other-transcode
+- other_video_transcoding-0.1.0.gem
+- other_video_transcoding-0.1.1.gem
+- other_video_transcoding-0.2.0.gem
+- other_video_transcoding-0.3.0.gem
+- other_video_transcoding-0.3.1.gem
+- other_video_transcoding-0.3.2.gem
+- other_video_transcoding-0.4.0.gem
+- other_video_transcoding-0.5.0.gem
 - other_video_transcoding.gemspec
 homepage: https://github.com/donmelton/other_video_transcoding
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -46,7 +54,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Other tools to transcode videos.
 test_files: []