Merge pull request #2 from nyu-dss/enhancement/smart-compress-tifs

Enhancement/smart compress tifs

Author: mnyrop

.rubocop.yml

@@ -15,13 +15,15 @@ Style/StringLiteralsInInterpolation:
   EnforcedStyle: single_quotes
 
 Layout/LineLength:
-  Max: 120
+  Max: 125
   AllowedPatterns:
     - 'raise'
 
 Naming/RescuedExceptionsVariableName:
   Enabled: false # conflicts with reek expectations
-
+Metrics/ClassLength:
+  Enabled: false
+  
 Metrics/BlockLength:
   Exclude:
     - '*.gemspec'

lib/aperitiiif.rb

@@ -2,6 +2,7 @@
 
 require 'colorize'
 require 'thor'
+require 'vips'
 
 require 'aperitiiif/asset'
 require 'aperitiiif/batch'

lib/aperitiiif/asset.rb

@@ -11,7 +11,8 @@ module Aperitiiif
   class Asset
     attr_reader :parent_id
 
-    TARGET_EXT = '.jpg'
+    TARGET_EXT  = '.tif'
+    TARGET_OPTS = { tile: :pyramid, compression: :jpeg, tile_width: 256, tile_height: 256 }.freeze
 
     def initialize(parent_id, source, config)
       @parent_id  = parent_id
@@ -93,7 +94,8 @@ def write_to_target
       return false if target_written?
 
       FileUtils.mkdir_p @config.image_build_dir
-      Vips::Image.new_from_file(@source).write_to_file target
+      Vips::Image.new_from_file(@source).tiffsave target, **TARGET_OPTS
+      GC.start
     end
 
     def service

lib/aperitiiif/utils.rb

@@ -11,7 +11,7 @@ def self.rm_ext(str)            = str.sub File.extname(str), ''
     def self.parent_id(str, dir)    = prune_prefix_junk(rm_ext(str.sub(dir, '')))
     def self.parent_dir(str)        = File.dirname(str).split('/').last
     def self.prune_prefix_junk(str) = str.sub(/^(_|\W+)/, '')
-    def self.mime(path) = MimeMagic.by_magic(File.open(path)).to_s
+    def self.mime(path) = MimeMagic.by_path(path).to_s
     def self.valid_source?(path) = File.file?(path) && path.end_with?(*ALLOWED_SRC_FORMATS)
 
     def self.slugify(str)

lib/aperitiiif/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Aperitiiif
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

spec/aperitiiif/asset_spec.rb

@@ -50,7 +50,7 @@
 
   describe '.target_mime' do
     it 'returns tiff mime string' do
-      expect(@asset.target_mime).to eq('image/jpeg')
+      expect(@asset.target_mime).to eq('image/tiff')
     end
   end