Merge pull request #374 from scientist-softserv/fix-image-processing

🐛 Fix image processing issue
notch8 · Nov 7, 2024 · 9c4783a · 9c4783a
2 parents 7f35ea9 + 28d5c7f
commit 9c4783a
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/lib/iiif_print/image_tool.rb b/lib/iiif_print/image_tool.rb
@@ -41,7 +41,7 @@ def convert_image(source, destination, monochrome)
       monochrome &&= destination.slice(-4, 4).index('tif')
       mono_opts = "-depth 1 -monochrome -compress Group4 -type bilevel "
       opts = monochrome ? mono_opts : ''
-      cmd = "convert #{source} #{opts}#{destination}"
+      cmd = "convert #{source} #{opts}#{destination} && sync"
       `#{cmd}`
     end
 
@@ -74,8 +74,11 @@ def im_identify_geometry(lines)
 
     # @return [Array<String>] lines of output from imagemagick `identify`
     def im_identify
-      cmd = "identify -format 'Geometry: %G\nDepth: %[bit-depth]\nColorspace: %[colorspace]\nAlpha: %A\nMIME type: %m\n' #{path}"
-      `#{cmd}`.lines
+      cmd = "identify -limit memory 8GiB -limit map 16GiB -limit disk 50GiB -format 'Geometry: %G\nDepth: %[bit-depth]\nColorspace: %[colorspace]\nAlpha: %A\nMIME type: %m\n' #{path}"
+      output, status = Open3.capture2(cmd)
+      Rails.logger.info "Identify command output: #{output}"
+      Rails.logger.info "Identify command status: #{status}"
+      output.lines
     end
 
     def im_mime(lines)