video_transcoding 0.17.2 → 0.17.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3724536547ba9854e753cbf28801016a3a9925f3
-  data.tar.gz: 618cc836292bae3f49052e5c126f266aa8a42324
+  metadata.gz: 241a0deb1c290faf3bc3369d8ffe5553d50fe625
+  data.tar.gz: 6f86fc711f6c6987a43e49e809de4c40bcfc28f2
 SHA512:
-  metadata.gz: aee6b5624993332b40a271eb82f47e2d8f678d8dd04a820a801ee42add82dbf695cc6fa76752ad2417339069d4d6aaf824923dfb6fbe5e608e9fe2037e45f616
-  data.tar.gz: 0e6ae2ff613b73c4c596b1dddcee1f3faadec8b3629eda4889b385973c5a6548cdcd750230c7ad62a61a19397111dc89300ecb8c02c5f5cf059f91af16154aaf
+  metadata.gz: 57641bd39a63881135fb6107dd1db91e2376b81c785a2a107b62ab22cd7b5226719c3681e8d6a3f4e9b541955ace2f00c2c66631af455941f4c892571eb4e179
+  data.tar.gz: 3413a920260f3526f5cfcf3d4c863bf8d010d293a6815cad2bc3411ffe5e52e19d92af77f8321e9efeb8fc90fefde8cbd6d19d4a4f1c47b9caed1619fa4a8c79

data/README.md

@@ -329,6 +329,8 @@ Which prints out something like this:
 
 Just copy and paste the sample commands to preview or transcode.
 
+Please note that path names within the sample commands are not escaped properly when using `cmd.exe` or PowerShell on Windows. If you have [Git for Windows](https://git-for-windows.github.io/) or another Unix-like environment installed, you can use the [Bash shell](https://en.wikipedia.org/wiki/Bash_(Unix_shell)) (usually named `bash.exe`) to work around this issue.
+
 If HandBrake and FFmpeg disagree about the cropping values, then `detect-crop` prints out something like this:
 
     Results differ...
@@ -660,6 +662,19 @@ For a few problematic videos, I have to apply options like `--force-rate 23.976
 
 ## History
 
+Sunday, May 14, 2017
+
+### [0.17.3](https://github.com/donmelton/video_transcoding/releases/tag/0.17.3)
+
+* `HandBrakeCLI` versions 1.0 and later changed the default frame rate mode from "constant" to "peak-limited" when a rate is specified. This new behavior in `HandBrakeCLI` requires two significant changes in `transcode-video`:
+    * Fix a bug where the `--force-rate` option failed to force a constant frame rate. This bug made it behave essentially the same at the `--limit-rate` option.
+    * Fix a bug where a constant frame rate was not forced for inputs containing [MPEG-2 video](https://en.wikipedia.org/wiki/MPEG-2). This bug affected the transcoding of all DVDs but very few Blu-ray Discs. The good news is that this bug probably didn't cause visual problems since the new default peak-limited implementation in `HandBrakeCLI` versions 1.0 and later worked like a constant frame rate most of the time.
+* Modify `convert-video` to use binary file mode when reading and writing console and log output from `ffmpeg`. This eliminates redundant information and "console spew" on Windows by suppressing the EOL <-> CRLF conversion. Thanks, [@samhutchins](https://github.com/samhutchins)! Via [ #147](https://github.com/donmelton/video_transcoding/pull/147).
+* Also modify `transcode-video` and `convert-video` to use binary file mode when processing console I/O from `mkvpropedit` and `mp4track` to eliminate that same "console spew" on Windows.
+* Modify `detect-crop` to escape preview commands for `cmd.exe` and PowerShell on Windows in a manner that's still compatible with Bourne and Z shells. Also mention in the "Using `detect-crop`" section of the "README" document that path names within the sample commands are not escaped properly when using `cmd.exe` or PowerShell on Windows and that `bash.exe` can be used as a workaround. Via [ #146](https://github.com/donmelton/video_transcoding/issues/146).
+* Modify `transcode-video` to accept `x264_10bit`, `x265_10bit`, `x265_12bit` and `x265_16bit` as supported encoders while also adjusting the encoder profile for these variants. Via [ #143](https://github.com/donmelton/video_transcoding/issues/143).
+* Modify `transcode-video` to no longer set the x264 encoder level if a frame rate has been requested higher than `30` FPS. Via [ #141](https://github.com/donmelton/video_transcoding/issues/141).
+
 ### [0.17.2](https://github.com/donmelton/video_transcoding/releases/tag/0.17.2)
 
 Monday, April 3, 2017

data/bin/convert-video

@@ -244,7 +244,7 @@ HERE
             '--track-index', track.to_s,
             '--enabled', enabled,
             output,
-          ], :err=>[:child, :out]) do |io|
+          ], 'rb', :err=>[:child, :out]) do |io|
             io.each do |line|
               Console.debug line
             end
@@ -385,7 +385,7 @@ HERE
           MKVpropedit.command_name,
           *mkvpropedit_options,
           output,
-        ], :err=>[:child, :out]) do |io|
+        ], 'rb', :err=>[:child, :out]) do |io|
           io.each do |line|
             Console.debug line
           end
@@ -410,7 +410,7 @@ HERE
       Console.info 'Converting with ffmpeg...'
 
       begin
-        IO.popen(ffmpeg_command, :err=>[:child, :out]) do |io|
+        IO.popen(ffmpeg_command, 'rb', :err=>[:child, :out]) do |io|
           Signal.trap 'INT' do
             Process.kill 'INT', io.pid
           end

data/bin/detect-crop

@@ -104,7 +104,7 @@ HERE
         puts
 
         if @player == :mpv
-          puts "mpv --no-audio --vf 'lavfi=[drawbox=#{Crop.drawbox_string(crop, width, height)}:invert:1]' #{shell_path}"
+          puts "mpv --no-audio --vf \"lavfi=[drawbox=#{Crop.drawbox_string(crop, width, height)}:invert:1]\" #{shell_path}"
           puts "mpv --no-audio --vf crop=#{Crop.player_string(crop, width, height)} #{shell_path}"
         else
           str = Crop.player_string(crop, width, height)

data/bin/transcode-video

@@ -451,6 +451,7 @@ HERE
         end
 
         force_handbrake_option 'rate', arg
+        force_handbrake_option 'cfr', nil
         @handbrake_options.delete 'vfr'
         @handbrake_options.delete 'pfr'
       end
@@ -908,11 +909,16 @@ HERE
         end
 
         handbrake_options['rate'] = rate
-        handbrake_options['pfr'] = nil if rate == '30'
+
+        if rate == '30'
+          handbrake_options['pfr'] = nil
+        else
+          handbrake_options['cfr'] = nil
+        end
       end
 
       encoder = @handbrake_options.fetch('encoder', 'x264')
-      return unless encoder == 'x264' or encoder == 'x265'
+      return unless encoder =~ /^x264(?:_10bit)?$/ or encoder =~ /^x265(?:_1[026]bit)?$/
       handbrake_options['encoder-preset'] = 'medium'
 
       if width > 1920 or height > 1080
@@ -932,9 +938,16 @@ HERE
       case encoder
       when 'x264'
         handbrake_options['encoder-profile'] = 'high'
-        handbrake_options['encoder-level'] = encoder_level
+
+        if @handbrake_options.fetch('rate', '30').to_f <= 30.0
+          handbrake_options['encoder-level'] = encoder_level
+        end
+      when 'x264_10bit'
+        handbrake_options['encoder-profile'] = 'high10'
       when 'x265'
         handbrake_options['encoder-profile'] = 'main'
+      when 'x265_10bit', 'x265_12bit', 'x265_16bit'
+        handbrake_options['encoder-profile'] = 'main10'
       end
 
       if @vbr_quality.nil?
@@ -972,7 +985,7 @@ HERE
 
       if  (@quick or @veryquick) and
           @handbrake_options.fetch('encoder-preset', 'medium') == 'medium' and
-          encoder == 'x264'
+          encoder =~ /^x264(?:_10bit)?$/
         encoder_options['partitions']   = 'none'
         encoder_options['ref']          = '1'
         encoder_options['bframes']      = '1'     if @veryquick
@@ -1306,7 +1319,7 @@ HERE
             '--edit', 'track:s1',
             '--set', 'flag-forced=1',
             output,
-          ], :err=>[:child, :out]) do |io|
+          ], 'rb', :err=>[:child, :out]) do |io|
             io.each do |line|
               Console.debug line
             end
@@ -1333,7 +1346,7 @@ HERE
                 '--edit', "track:a#{track}",
                 '--set', "name=#{original_name}",
                 output,
-              ], :err=>[:child, :out]) do |io|
+              ], 'rb', :err=>[:child, :out]) do |io|
                 io.each do |line|
                   Console.debug line
                 end
@@ -1353,7 +1366,7 @@ HERE
                   '--track-index', track.to_s,
                   "--#{property}", original_name,
                   output,
-                ], :err=>[:child, :out]) do |io|
+                ], 'rb', :err=>[:child, :out]) do |io|
                   io.each do |line|
                     Console.debug line
                   end

data/lib/video_transcoding/version.rb

@@ -5,5 +5,5 @@
 #
 
 module VideoTranscoding
-  VERSION = '0.17.2'
+  VERSION = '0.17.3'
 end

metadata

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