image_processing 1.10.1 → 1.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5141a7684e311666d6a8a91bf74cd791757df4bafe8b989f42c4785aa7bd2bf
-  data.tar.gz: 2639145e388bf411cbf03559151c12a5eec8757e96b6bd52ae993533e7559781
+  metadata.gz: ec4b4776480756b2721185419de81f228e5ca927b4bd3077fb4c0a5dcafef15f
+  data.tar.gz: d40b8822b6a7b2b5e1bad182099f7e3b23b25d0cdd6b61b1afa3154b665e76a6
 SHA512:
-  metadata.gz: aa03273ffdb5d53a8cd944921db4e6359fb198e308373a15618c4c824cbe981e41c40544f7b64f1008175de30d2ea60f92cfb346564f1d4166e164d27759c4af
-  data.tar.gz: febc2324ceccb4b0fca18c708ad3c2423239fa458065cb74790a574e8149acc6285709d356eb99ec2896611803cc536707b47c1b3b3c68b3067377fb881dffb0
+  metadata.gz: 77cff245646f56f409d4c1d006d26cee819d11cc9c263b88d86bafb6e480ac691240bfa17542bf9c578891056355d53f63fefddfa1f3eb801e3ddeea20666ae3
+  data.tar.gz: bf3f39c4204d7dcbb6382596a4a8850f8c7d22fee94e8ed77a1782a58bd1f370448268bb4e74028e429ee2e7c6156613b1022f5465faf6d30ab01833751c4df9

data/CHANGELOG.md

@@ -1,3 +1,27 @@
+## 1.12.1 (2020-11-06)
+
+* Fix format fallback for files ending with a dot on Ruby 2.7+ (@coding-chimp)
+
+## 1.12.0 (2020-09-20)
+
+* Add instrumentation support via `#instrumenter` (@janko)
+
+## 1.11.0 (2020-05-17)
+
+* [minimagick] Handle destination format having no file extension (@janko)
+
+* [minimagick] Disable sharpening on `#resize_*` operators by default (@flori)
+
+* [minimagick] Add `#crop` which accepts `left, top, width, height` arguments (@janko)
+
+## 1.10.3 (2020-01-12)
+
+* [vips] Fix auto-rotation not working in certain cases on libvips 8.9.0 (@janko)
+
+## 1.10.2 (2020-01-11)
+
+* Fix Ruby 2.7 warnings for separation of positional and keyword arguments (@kamipo, @janko)
+
 ## 1.10.1 (2020-01-07)
 
 * [vips] Fix compatibility with ruby-vips 2.0.17+ (@janko)

data/README.md

@@ -148,6 +148,34 @@ You can continue reading the API documentation for specific modules:
 See the **[wiki]** for additional "How To" guides for common scenarios. The wiki
 is publicly editable, so you're encouraged to add your own guides.
 
+## Instrumentation
+
+You can register an `#instrumenter` block for a given pipeline, which will wrap
+the pipeline execution, allowing you to record performance metrics.
+
+```rb
+pipeline = ImageProcessing::Vips.instrumenter do |**options, &processing|
+  options[:source]     #=> #<File:...>
+  options[:loader]     #=> { fail: true }
+  options[:saver]      #=> { quality: 85 }
+  options[:format]     #=> "png"
+  options[:operations] #=> [[:resize_to_limit, 500, 500], [:flip, [:horizontal]]]
+  options[:processor]  #=> ImageProcessing::Vips::Processor
+
+  ActiveSupport::Notifications.instrument("process.image_processing", **options) do
+    processing.call # calls the pipeline
+  end
+end
+
+pipeline
+  .source(image)
+  .loader(fail: true)
+  .saver(quality: 85)
+  .convert("png")
+  .resize_to_limit(500, 500)
+  .flip(:horizontal)
+  .call # calls instrumenter
+```
 
 ## Contributing
 

data/image_processing.gemspec

@@ -6,8 +6,8 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = ">= 2.3"
 
-  spec.summary       = "Set of higher-level helper methods for image processing."
-  spec.description   = "Set of higher-level helper methods for image processing."
+  spec.summary       = "High-level wrapper for processing images for the web with ImageMagick or libvips."
+  spec.description   = "High-level wrapper for processing images for the web with ImageMagick or libvips."
   spec.homepage      = "https://github.com/janko/image_processing"
   spec.authors       = ["Janko Marohnić"]
   spec.email         = ["janko.marohnic@gmail.com"]

data/lib/image_processing/builder.rb

@@ -9,8 +9,24 @@ module ImageProcessing
     end
 
     # Calls the pipeline to perform the processing from built options.
-    def call!(**options)
-      Pipeline.new(self.options).call(**options)
+    def call!(**call_options)
+      instrument do
+        Pipeline.new(pipeline_options).call(**call_options)
+      end
+    end
+
+    private
+
+    def instrument
+      return yield unless options[:instrumenter]
+
+      result = nil
+      options[:instrumenter].call(**pipeline_options) { result = yield }
+      result
+    end
+
+    def pipeline_options
+      options.reject { |key, _| key == :instrumenter }
     end
   end
 end

data/lib/image_processing/chainable.rb

@@ -21,6 +21,11 @@ module ImageProcessing
       branch saver: options
     end
 
+    # Register instrumentation block that will be called around the pipeline.
+    def instrumenter(&block)
+      branch instrumenter: block
+    end
+
     # Add multiple operations as a hash or an array.
     #
     #   .apply(resize_to_limit: [400, 400], strip: true)
@@ -32,21 +37,14 @@ module ImageProcessing
           builder.send(name)
         elsif argument.is_a?(Array)
           builder.send(name, *argument)
+        elsif argument.is_a?(Hash)
+          builder.send(name, **argument)
         else
           builder.send(name, argument)
         end
       end
     end
 
-    # Assume that any unknown method names an operation supported by the
-    # processor. Add a bang ("!") if you want processing to be performed.
-    def method_missing(name, *args, &block)
-      return super if name.to_s.end_with?("?")
-      return send(name.to_s.chomp("!"), *args, &block).call if name.to_s.end_with?("!")
-
-      operation(name, *args, &block)
-    end
-
     # Add an operation defined by the processor.
     def operation(name, *args, &block)
       branch operations: [[name, args, *block]]
@@ -55,27 +53,41 @@ module ImageProcessing
     # Call the defined processing and get the result. Allows specifying
     # the source file and destination.
     def call(file = nil, destination: nil, **call_options)
-      options = {}
-      options = options.merge(source: file) if file
-      options = options.merge(destination: destination) if destination
+      options = { source: file, destination: destination }.compact
 
-      branch(options).call!(**call_options)
+      branch(**options).call!(**call_options)
     end
 
     # Creates a new builder object, merging current options with new options.
-    def branch(loader: nil, saver: nil, operations: nil, **other_options)
-      options = respond_to?(:options) ? self.options : DEFAULT_OPTIONS
+    def branch(**new_options)
+      if self.is_a?(Builder)
+        options = self.options
+      else
+        options = DEFAULT_OPTIONS.merge(processor: self::Processor)
+      end
 
-      options = options.merge(loader: options[:loader].merge(loader)) if loader
-      options = options.merge(saver: options[:saver].merge(saver)) if saver
-      options = options.merge(operations: options[:operations] + operations) if operations
-      options = options.merge(processor: self::Processor) unless self.is_a?(Builder)
-      options = options.merge(other_options)
+      options = options.merge(new_options) do |key, old_value, new_value|
+        case key
+        when :loader, :saver then old_value.merge(new_value)
+        when :operations     then old_value + new_value
+        else                      new_value
+        end
+      end
+
+      Builder.new(options.freeze)
+    end
 
-      options.freeze
+    private
 
-      Builder.new(options)
+    # Assume that any unknown method names an operation supported by the
+    # processor. Add a bang ("!") if you want processing to be performed.
+    def method_missing(name, *args, &block)
+      return super if name.to_s.end_with?("?")
+      return send(name.to_s.chomp("!"), *args, &block).call if name.to_s.end_with?("!")
+
+      operation(name, *args, &block)
     end
+    ruby2_keywords(:method_missing) if respond_to?(:ruby2_keywords, true)
 
     # Empty options which the builder starts with.
     DEFAULT_OPTIONS = {

data/lib/image_processing/mini_magick.rb

@@ -32,7 +32,7 @@ module ImageProcessing
           source_path = path_or_magick
           magick = ::MiniMagick::Tool::Convert.new
 
-          Utils.apply_options(magick, options)
+          Utils.apply_options(magick, **options)
 
           input  = source_path
           input  = "#{loader}:#{input}" if loader
@@ -50,7 +50,7 @@ module ImageProcessing
       # the result to disk. Accepts additional options related to saving the
       # image (e.g. quality).
       def self.save_image(magick, destination_path, allow_splitting: false, **options)
-        Utils.apply_options(magick, options)
+        Utils.apply_options(magick, **options)
 
         magick << destination_path
         magick.call
@@ -86,6 +86,15 @@ module ImageProcessing
         magick.extent "#{width}x#{height}"
       end
 
+      # Crops the image with the specified crop points.
+      def crop(*args)
+        case args.count
+        when 1 then magick.crop(*args)
+        when 4 then magick.crop("#{args[2]}x#{args[3]}+#{args[0]}+#{args[1]}")
+        else fail ArgumentError, "wrong number of arguments (expected 1 or 4, got #{args.count})"
+        end
+      end
+
       # Rotates the image by an arbitrary angle. For angles that are not
       # multiple of 90 degrees an optional background color can be specified to
       # fill in the gaps.
@@ -161,7 +170,7 @@ module ImageProcessing
 
       # Resizes the image using the specified geometry, and sharpens the
       # resulting thumbnail.
-      def thumbnail(geometry, sharpen: {})
+      def thumbnail(geometry, sharpen: nil)
         magick.resize(geometry)
 
         if sharpen
@@ -192,7 +201,7 @@ module ImageProcessing
         # format, ImageMagick will create multiple images, one for each layer.
         # We want to warn the user that this is probably not what they wanted.
         def disallow_split_layers!(destination_path)
-          layers = Dir[destination_path.sub(/\.\w+$/, '-*\0')]
+          layers = Dir[destination_path.sub(/(\.\w+)?$/, '-*\0')]
 
           if layers.any?
             layers.each { |path| File.delete(path) }

data/lib/image_processing/pipeline.rb

@@ -37,9 +37,9 @@ module ImageProcessing
 
     # Determines the appropriate destination image format.
     def destination_format
-      format   = File.extname(destination)[1..-1] if destination
+      format   = determine_format(destination) if destination
       format ||= self.format
-      format ||= File.extname(source_path)[1..-1] if source_path
+      format ||= determine_format(source_path) if source_path
 
       format || DEFAULT_FORMAT
     end
@@ -93,5 +93,11 @@ module ImageProcessing
         @source
       end
     end
+
+    def determine_format(file_path)
+      extension = File.extname(file_path)
+
+      extension[1..-1] if extension.size > 1
+    end
   end
 end

data/lib/image_processing/processor.rb

@@ -34,19 +34,12 @@ module ImageProcessing
       const_set(:ACCUMULATOR_CLASS, klass)
     end
 
-    # Calls the operation to perform the processing. If the operation is
-    # defined on the processor (macro), calls it. Otherwise calls the
-    # operation directly on the accumulator object. This provides a common
-    # umbrella above defined macros and direct operations.
+    # Delegates to #apply_operation.
     def self.apply_operation(accumulator, (name, args, block))
-      if method_defined?(name)
-        instance = new(accumulator)
-        instance.public_send(name, *args, &block)
-      else
-        accumulator.send(name, *args, &block)
-      end
+      new(accumulator).apply_operation(name, *args, &block)
     end
 
+    # Whether the processor supports resizing the image upon loading.
     def self.supports_resize_on_load?
       false
     end
@@ -55,6 +48,21 @@ module ImageProcessing
       @accumulator = accumulator
     end
 
+    # Calls the operation to perform the processing. If the operation is
+    # defined on the processor (macro), calls the method. Otherwise calls the
+    # operation directly on the accumulator object. This provides a common
+    # umbrella above defined macros and direct operations.
+    def apply_operation(name, *args, &block)
+      receiver = respond_to?(name) ? self : @accumulator
+
+      if args.last.is_a?(Hash)
+        kwargs = args.pop
+        receiver.public_send(name, *args, **kwargs, &block)
+      else
+        receiver.public_send(name, *args, &block)
+      end
+    end
+
     # Calls the given block with the accumulator object. Useful for when you
     # want to access the accumulator object directly.
     def custom(&block)

data/lib/image_processing/version.rb

@@ -1,3 +1,3 @@
 module ImageProcessing
-  VERSION = "1.10.1"
+  VERSION = "1.12.1"
 end

data/lib/image_processing/vips.rb

@@ -138,8 +138,13 @@ module ImageProcessing
           # resize on load
           image = ::Vips::Image.thumbnail(self.image, width, height: height, **options)
         else
+          # we're already calling Image#autorot when loading the image
+          no_rotate = ::Vips.at_least_libvips?(8, 8) ? { no_rotate: true } : { auto_rotate: false }
+          options   = no_rotate.merge(options)
+
           image = self.image.thumbnail_image(width, height: height, **options)
         end
+
         image = image.conv(sharpen, precision: :integer) if sharpen
         image
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: image_processing
 version: !ruby/object:Gem::Version
-  version: 1.10.1
+  version: 1.12.1
 platform: ruby
 authors:
 - Janko Marohnić
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-07 00:00:00.000000000 Z
+date: 2020-11-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mini_magick
@@ -120,7 +120,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Set of higher-level helper methods for image processing.
+description: High-level wrapper for processing images for the web with ImageMagick
+  or libvips.
 email:
 - janko.marohnic@gmail.com
 executables: []
@@ -158,8 +159,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
-summary: Set of higher-level helper methods for image processing.
+summary: High-level wrapper for processing images for the web with ImageMagick or
+  libvips.
 test_files: []