video_transcoding 0.10.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f473590263e13e9651405d29aac6923931e2e7ce
-  data.tar.gz: c106e10d026d8f784e616da4cc983e3ac7fae120
+  metadata.gz: 1ba201ee60bae63940461ad235187b738723c478
+  data.tar.gz: c1e50ded6787be60122559915853a1cdf38fe4df
 SHA512:
-  metadata.gz: ab2819953f8ad74407948cc485c4b38eee6a83df9d55c26b34f4f3bd77115b8c7f28f5682b56ce1dcde651410c4c33b78505e2da10a34e440f0918c9931da8b1
-  data.tar.gz: 25a1feffd51663c3d7dc8e1e98c6b3516ff277c2b63168a134deda5b0ac80f7841541389cbedb5df508b744e395b2e4fde497d112257038ca2c05298a0cf860a
+  metadata.gz: 2d57ffe063543ec6651c756a1597b423f4e8c196e29818a7e97fba315c433787363b27d8be0063bdbdd9372c0547bf13c000835e09b36a1f48fa58b5fa175171
+  data.tar.gz: 6f2e1b09bc9bc9e4dfc4dfbb10843c089c5375ed739d35388821fb5bd838e9ba54e4180a0c02571c32f228c9f8ff3dde88ab383289f1ffc1a51b369dfd9a6308

data/README.md

@@ -16,7 +16,7 @@ Most of the tools in this package are essentially intelligent wrappers around Op
 Transcode video file or disc image directory into format and size similar to popular online downloads.
 
 * [`detect-crop`](#why-detect-crop)
-Detect optimal crop values for video file or disc image directory.
+Detect crop values for video file or disc image directory.
 
 * [`convert-video`](#why-convert-video)
 Convert video file from Matroska to MP4 format or from MP4 to Matroksa format without transcoding video.
@@ -117,17 +117,11 @@ But most of these default settings and automatic behaviors can be easily overrid
 
 ### Why `detect-crop`?
 
-Removing the black, non-content borders of a video during transcoding is not about making the edges of the output look pretty. Those edges are usually not visible anyway when viewed full screen.
-
-Cropping is about faster transcoding and higher quality. Fewer pixels to read and write almost always leads to a speed improvement. Fewer pixels also means the x264 encoder within HandBrake doesn't waste bitrate on non-content.
-
 HandBrake applies automatic crop detection by default. While it's usually correct, it does guess wrong often enough not to be trusted without review. For example, HandBrake's default behavior removes the top and bottom 140 pixels from "[The Dark Knight (2008)](http://www.blu-ray.com/movies/The-Dark-Knight-Blu-ray/743/)" and "[The Hunger Games: Catching Fire (2013)](http://www.blu-ray.com/movies/The-Hunger-Games-Catching-Fire-Blu-ray/67923/)," losing significant portions of their full-frame content.
 
-And sometimes HandBrake only crops a few pixels from one or more edges, which is too small of a difference in size to improve performance or quality.
-
 This is why `transcode-video` doesn't allow HandBrake to apply cropping by default.
 
-Instead, the `detect-crop` tool leverages both HandBrake and MPlayer, with additional measurements and constraints, to find the optimal video cropping bounds. It then indicates whether those two programs agree. To aid in review, this tool prints commands to the terminal console allowing the recommended (or disputed) crop to be displayed, as well as a sample command line for `transcode-video` itself.
+Instead, the `detect-crop` tool leverages both HandBrake and MPlayer to find the video cropping bounds. It then indicates whether those two programs agree. To aid in review, this tool prints commands to the terminal console allowing the recommended (or disputed) crop to be displayed, as well as a sample command line for `transcode-video` itself.
 
 ### Why `convert-video`?
 
@@ -201,7 +195,7 @@ The `.m4v` file extension is more "iTunes-friendly," but the file content itself
 
 #### Reducing output size
 
-If reducing output size is more important to you than quality, use the `--small` option:
+If reducing output size is more important to you than a possible loss in quality, use the `--small` option:
 
     transcode-video --small "/path/to/Movie.mkv"
 
@@ -233,7 +227,7 @@ The `--quick` option is also more than 15% speedier than the x264 video encoder'
 
 Be aware that output files are slightly larger when using the `--quick` option since the loss of precision is also a loss of efficiency.
 
-Performance also improves slightly using the `--small` option due to fewer calculations being made and fewer bits being written to disk.
+Performance also improves using the `--small` option due to fewer calculations being made and fewer bits being written to disk.
 
 #### Cropping
 
@@ -247,7 +241,7 @@ This command removes the left and right 240 pixels, typical of a 4:3 classic TV
 
     transcode-video --crop 0:0:240:240 "/path/to/Movie.mkv"
 
-Use the `detect-crop` tool to determine the optimal cropping bounds.
+Use the `detect-crop` tool to determine the cropping bounds before transcoding.
 
 You can also call the `detect-crop` logic from `transcode-video` with the single `detect` argument:
 
@@ -343,7 +337,7 @@ Unlike `HandBrakeCLI`, external subtitle file names are allowed to contain comma
 
 ### Using `detect-crop`
 
-The command to find the optimal video cropping bounds is as simple as:
+The command to find the video cropping bounds is as simple as:
 
     detect-crop "/path/to/Movie.mkv"
 
@@ -378,8 +372,6 @@ You'll then need to preview both and decide which to use.
 
 When input is a disc image directory instead of a single file, the `detect-crop` tool doesn't use MPlayer, nor does it print out commands to preview the crop.
 
-Be aware that the algorithm to determine optimal shape always crops from the top and bottom or from the left and right, never from both axes.
-
 ### Using `convert-video`
 
 The `convert-video` tool repackages video files, converting them from Matroska to MP4 format or from MP4 to Matroksa format without transcoding the video. It's as simple as:
@@ -504,11 +496,11 @@ Use the default settings whenever possible.
 
 Use the `--mp4` or `--m4v` options if your target player can't handle Matroska format.
 
-Use the `--small` option if you're short on storage space.
+Use the `--small` option if you want more space savings.
 
 Use the `--quick` option if you're in a hurry.
 
-Apply unambiguous crop values from `detect-crop` after review.
+Use `detect-crop` before transcoding to manually review and apply the best crop values.
 
 Don't add audio tracks in their original format that aren't AAC or Dolby Digital AC-3.
 
@@ -631,16 +623,31 @@ What I don't use are [peak signal-to-noise ratios](https://en.wikipedia.org/wiki
 
 ### What options do you use with `transcode-video`?
 
-I use the default settings. That's why they're the defaults.
+Instead of the default size, I use the `--small` option for more space savings.
+
+I never use the `--crop detect` function of `transcode-video` because I don't trust either `HandBrakeCLI` or `mplayer` to always get it right without supervision. Instead, I use the separate `detect-crop` tool before transcoding to manually review and apply the best crop values.
 
-Of course, I apply crop values as necessary. And I let `transcode-video` automatically burn any forced subtitles into the output video track when the "forced" flag is enabled in the original.
+I let `transcode-video` automatically burn any forced subtitles into the output video track when the "forced" flag is enabled in the original.
 
 I never include separate subtitle tracks, but I do add audio commentary tracks.
 
-For a few problematic videos, I have to apply options like `--force-rate 23.976 --filter detelecine`. But that's rare.
+For a few problematic videos, I have to apply options like `--force-rate 23.976 --filter detelecine`. For others, options like `--encoder-option zones=...` to adjust bitrate distribution. But both of these cases are rare.
 
 ## History
 
+### [0.11.0](https://github.com/donmelton/video_transcoding/releases/tag/0.11.0)
+
+Thursday, September 15, 2016
+
+* Change the behavior of `detect-crop` and the `--crop detect` function of `transcode-video` to no longer constrain the crop by default. Add a `--constrain` option to `detect-crop` and a `--constrain-crop` option to `transcode-video` to restore the old behavior. Also, deprecate the `--no-constrain` option of `detect-crop` and the `--no-constrain-crop` option of `transcode-video` since both are no longer necessary. Via [ #81](https://github.com/donmelton/video_transcoding/issues/81).
+* Update the "README" document to:
+    * Revise multiple sections about the changes to cropping behavior.
+    * Revise the description of the `--small` option in multiple sections.
+    * Revise how I use `transcode-video` in the "FAQ" section.
+* Add support for the `comb-detect`, `hqdn3d` and `pad` filters to `transcode-video`.
+* Fix a bug in `transcode-video` where the `--filter` option failed when `nlmeans-tune` was used as a argument. This was due to a regular expression only allowing lowercase alpha characters and not hyphens.
+* Update the default AC-3 audio and pass-through bitrates in the `--help` output of `transcode-video` to 640 Kbps, matching the behavior of the code since version 0.5.0.
+
 ### [0.10.0](https://github.com/donmelton/video_transcoding/releases/tag/0.10.0)
 
 Friday, May 6, 2016

data/bin/detect-crop

@@ -22,14 +22,14 @@ HERE
 
     def usage
       <<HERE
-Detect optimal crop values for video file or disc image directory.
+Detect crop values for video file or disc image directory.
 
 Usage: #{$PROGRAM_NAME} [OPTION]... [FILE|DIRECTORY]...
 
     --scan          list title(s) and tracks in video media and exit
     --title NUMBER  select numbered title in video media
                       (default: main feature or first listed)
-    --no-constrain  don't constrain crop to optimal shape
+    --constrain     constrain crop to optimal shape
     --values-only   output only unambiguous crop values, not commands
 
 -v, --verbose       increase diagnostic information
@@ -46,15 +46,20 @@ HERE
       super
       @scan         = false
       @title        = nil
-      @constrain    = true
+      @constrain    = false
       @values_only  = false
     end
 
     def define_options(opts)
       opts.on('--scan')               { @scan = true }
       opts.on('--title ARG', Integer) { |arg| @title = arg }
-      opts.on('--no-constrain')       { @constrain = false }
+      opts.on('--constrain')          { @constrain = true }
       opts.on('--values-only')        { @values_only = true }
+
+      opts.on '--no-constrain' do
+        Console.warn 'Using deprecated option: --no-constrain'
+        @constrain = false
+      end
     end
 
     def configure

data/bin/transcode-video

@@ -67,10 +67,10 @@ Quality options:
 
 Video options:
     --crop T:B:L:R  set video crop values (default: 0:0:0:0)
-                      (use `--crop detect` for optimal crop values)
+                      (use `--crop detect` for `detect-crop` behavior)
                       (use `--crop auto` for `HandBrakeCLI` behavior)
-    --no-constrain-crop
-                    don't constrain `--crop detect` to optimal shape
+    --constrain-crop
+                    constrain `--crop detect` to optimal shape
     --fallback-crop handbrake|mplayer|none
                     select fallback crop values if `--crop detect` fails
     --720p          fit video within 1280x720 pixel bounds
@@ -121,9 +121,9 @@ Audio options:
                       with `stereo` to allow only single stereo track
                       (can be used multiple times)
     --ac3-bitrate 384|448|640
-                    set AC-3 audio bitrate (default: 384)
+                    set AC-3 audio bitrate (default: 640)
     --pass-ac3-bitrate 384|448|640
-                    set AC-3 audio pass-through bitrate (default: 448)
+                    set AC-3 audio pass-through bitrate (default: 640)
     --copy-audio TRACK|all
                     try to copy track selected by number in its original format
                       falling back to AC-3 format if original not allowed
@@ -225,7 +225,7 @@ HERE
       @vbv_maxrate_480p           = 2000
       @quick                      = false
       @crop                       = {:top => 0, :bottom => 0, :left => 0, :right => 0}
-      @constrain_crop             = true
+      @constrain_crop             = false
       @fallback_crop              = nil
       @target_bitrate             = nil
       @main_audio                 = nil
@@ -354,7 +354,12 @@ HERE
         end
       end
 
+      opts.on '--constrain-crop' do
+        @constrain_crop = true
+      end
+
       opts.on '--no-constrain-crop' do
+        Console.warn 'Using deprecated option: --no-constrain-crop'
         @constrain_crop = false
       end
 
@@ -449,10 +454,11 @@ HERE
       end
 
       opts.on '--filter ARG' do |arg|
-        if arg =~ /^([a-z]+)(?:=(.+))?$/
+        if arg =~ /^([a-z0-9-]+)(?:=(.+))?$/
           case $1
-          when 'deinterlace', 'decomb', 'detelecine', 'denoise', 'nlmeans',
-          'nlmeans-tune', 'deblock', 'rotate', 'grayscale'
+          when 'comb-detect', 'deinterlace', 'decomb', 'detelecine',
+          'hqdn3d', 'denoise', 'nlmeans', 'nlmeans-tune', 'deblock',
+          'rotate', 'pad', 'grayscale'
             force_handbrake_option $1, $2
           else
             fail UsageError, "unsupported filter name: #{$1}"

data/lib/video_transcoding/version.rb

@@ -5,5 +5,5 @@
 #
 
 module VideoTranscoding
-  VERSION = '0.10.0'
+  VERSION = '0.11.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: video_transcoding
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Don Melton
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-06 00:00:00.000000000 Z
+date: 2016-09-16 00:00:00.000000000 Z
 dependencies: []
 description: |2
       Video Transcoding is a package of tools to transcode, inspect