mork 0.18.0 → 0.18.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 46d443ec81a0831a6048e8a7a7b6a16eb35d6a7d9a1a0a13287427d44d7e9b77
-  data.tar.gz: 15788c1658c1c07dce5fc1e0c66a345afb1a15a28a3b85bca787437a6500b13e
+  metadata.gz: 8cdb6af94ff682ccad23d669fb624d263ea4fb28d5911e3ada2ceeb200c8fd4a
+  data.tar.gz: 956342c055f892ee7e4c44187711b3660eefd8fcee68723bfd1cb2fb245d0a75
 SHA512:
-  metadata.gz: cd65ed345abadaad63c283965c49c9af3c5b7b57564a8f0cabae07bc7d5bc32e8c167156c5703ad03b1b2ea4ff6bb0b363f976258589b2d37639cc3a39e4c1cc
-  data.tar.gz: bf1ae484be720c8a60f4e4f2cb540d5f9e235f89fd772089cb5bd5968762d54a3a1a1e4fb99725c29653633dbbec1732e3d8da953009fc583e4d734e86665a76
+  metadata.gz: 0c75eaac5ad245b5c86dde97f7e68c43a08486c248014e27963c8c1f763b2cc14fbe545d13f1d75675f1367daaf11df05f8109fdf88d752324cf7972a560e9e4
+  data.tar.gz: 8864848f8f6a4a9cada44dd6f74183928fcdbf15986c97a27eb4d95ba6f60f2c9f0dd6a95ec8029fe961df0f4289da21703e8b9472c4ba6ad3f70929c043ffeb

data/lib/mork/magicko.rb

@@ -18,7 +18,11 @@ module Mork
       # a default of 150 dpi seems sensible. It should not affect bitmaps.
       density = 150
       # inspect the source file
-      s1, s2, s3 = Open3.capture3 "identify -density #{density} -format '%w %h %m' #{path}"
+      identify = MiniMagick.identify
+      identify.density density
+      identify.format '%w %h %m'
+      identify << path
+      s1, s2, s3 = Open3.capture3(*identify.command)
       if s3.success?
         # parse the identify command output
         w, h, @type = s1.split(' ')
@@ -54,16 +58,17 @@ module Mork
     # (i.e. the centers of the four registration marks)
     # pp: a hash in the form of pp[:tl][:x], pp[:tl][:y], etc.
     def registered_bytes(pp)
-      read_bytes "-distort Perspective '#{pps pp}'"
+      read_bytes '-distort', 'Perspective', pps(pp)
     end
 
     # Reading from the image file the bytes from one of the four corner
     # squares encompassing each registration mark; the blur and dilation
     # manipulations may prevent registration misalignments due to stray dark pixels
     def rm_patch(c, blr=0, dlt=0)
-      b = blr==0 ? '' : " -blur #{blr*3}x#{blr}"
-      d = dlt==0 ? '' : " -morphology Dilate Octagon:#{dlt}"
-      read_bytes "-crop #{c.cropper}#{b}#{d}"
+      params = ['-crop', c.cropper]
+      params.concat ['-blur', "#{blr*3}x#{blr}"] unless blr == 0
+      params.concat ['-morphology', 'Dilate', "Octagon:#{dlt}"] unless dlt == 0
+      read_bytes(*params)
     end
 
     ##################################
@@ -171,10 +176,18 @@ module Mork
 
     # calling imagemagick and capturing the converted image
     # into an array of bytes
-    def read_bytes(params=nil)
-      d = @density ? "-density #{@density}" : nil
-      cmd = "magick -depth 8 #{d} #{@path} #{params} gray:-"
-      stdout, stderr, status = Open3.capture3(cmd)
+    def read_bytes(*params)
+      # MiniMagick selects `convert` on ImageMagick 6 and `magick` on
+      # ImageMagick 7. Execute its argv directly to preserve binary stdout
+      # (MiniMagick::Tool#call strips a trailing newline) and avoid a shell.
+      command = MiniMagick.convert
+      command.depth 8
+      command.density @density if @density
+      command << @path
+      command.merge!(params)
+      command << 'gray:-'
+
+      stdout, stderr, status = Open3.capture3(*command.command)
       if status.success?
         stdout.unpack('C*')
       else

data/lib/mork/sheet_pdf.rb

@@ -8,7 +8,8 @@ module Mork
   class SheetPDF < Prawn::Document
     DEFAULT_FONT_FAMILY = 'Open Sans'
     DEFAULT_FONT_PATH = File.expand_path('fonts/OpenSans-Regular.ttf', __dir__)
-    private_constant :DEFAULT_FONT_FAMILY, :DEFAULT_FONT_PATH
+    ITEM_TEXT_RISE_EM = 0.177
+    private_constant :DEFAULT_FONT_FAMILY, :DEFAULT_FONT_PATH, :ITEM_TEXT_RISE_EM
 
     include Extensions
     def initialize(content, layout=nil, duplex=false)
@@ -142,7 +143,7 @@ module Mork
     def question_numbers
       ch_len.first.length.times do |i|
         text_box "#{i+1}",
-                 at: @grip.qnum_xy(i),
+                 at: item_text_at(@grip.qnum_xy(i)),
                  width: @grip.qnum_width,
                  height: @grip.height_of_cell,
                  align: :right,
@@ -176,7 +177,7 @@ module Mork
                                    @grip.width_of_uid,
                                    @grip.height_of_uid,
                                    @grip.uround
-          text_box i, at: [offx, 0],
+          text_box i, at: item_text_at([offx, 0]),
                       width: @grip.width_of_uid,
                       height: @grip.height_of_uid,
                       align: :center,
@@ -201,13 +202,20 @@ module Mork
                                @grip.height_of_cell,
                                @grip.cround
       text_box l,
-               at:     [x,0],
+               at:     item_text_at([x, 0]),
                width:  @grip.width_of_cell,
                height: @grip.height_of_cell,
                align:  :center,
                valign: :center
     end
 
+    # Prawn vertically centers the font's full ascender box. Open Sans has a
+    # tall ascender relative to its visible capitals and digits, so raise only
+    # item text while leaving the OMR geometry unchanged.
+    def item_text_at(position)
+      [position[0], position[1] + @grip.item_font_size * ITEM_TEXT_RISE_EM]
+    end
+
     def equal_choice_number?
       return @equal_choice_number unless @equal_choice_number.nil?
 

data/lib/mork/version.rb

@@ -1,3 +1,3 @@
 module Mork
-  VERSION = '0.18.0'
+  VERSION = '0.18.2'
 end

data/spec/mork/magicko_spec.rb

@@ -39,6 +39,30 @@ module Mork
       end
     end
 
+    describe 'ImageMagick command selection' do
+      let(:status) { instance_double(Process::Status, success?: true) }
+
+      it 'uses ImageMagick 6 convert through MiniMagick and preserves raw bytes' do
+        magicko = ma
+        allow(MiniMagick).to receive(:imagemagick7?).and_return(false)
+        expect(Open3).to receive(:capture3).with(
+          'convert', '-depth', '8', sh.image_path, '-crop', co.cropper, 'gray:-'
+        ).and_return(["\x00\x0a\xff".b, '', status])
+
+        expect(magicko.rm_patch(co)).to eq [0, 10, 255]
+      end
+
+      it 'uses ImageMagick 7 magick through MiniMagick' do
+        magicko = ma
+        allow(MiniMagick).to receive(:imagemagick7?).and_return(true)
+        expect(Open3).to receive(:capture3).with(
+          'magick', '-depth', '8', sh.image_path, '-crop', co.cropper, 'gray:-'
+        ).and_return(["\x00".b, '', status])
+
+        expect(magicko.rm_patch(co)).to eq [0]
+      end
+    end
+
     describe 'overlay stroke width' do
       it 'defaults to the legacy 3-pixel width' do
         expect(ma.overlay_stroke_width).to eq 3

data/spec/mork/sheet_pdf_spec.rb

@@ -68,6 +68,18 @@ module Mork
       expect(bytes).to include 'OpenSans-Regular'
     end
 
+    it 'raises item labels without changing their OMR coordinates' do
+      pdf = SheetPDF.new({})
+      grid = pdf.instance_variable_get('@grip')
+      cell_position = grid.item_xy(0)
+
+      text_position = pdf.send(:item_text_at, cell_position)
+
+      expect(text_position[0]).to eq cell_position[0]
+      expect(text_position[1] - cell_position[1]).to be_within(0.001).of(1.593)
+      expect(grid.item_xy(0)).to eq cell_position
+    end
+
     it 'creates a minimal PDF sheet' do
       s = SheetPDF.new({})
       s.save dest 'minimal'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mork
 version: !ruby/object:Gem::Version
-  version: 0.18.0
+  version: 0.18.2
 platform: ruby
 authors:
 - Giuseppe Bertini