imogen 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 032e49d9a8602410ab24b57efc8bd3de1cd523ad41e217bd76affcca385a08aa
-  data.tar.gz: 2ae67a08b37695662dd21af0eedc7c7449423fde11ca20cd02c2f66e00ade5f4
+  metadata.gz: b1d474982ff7cb978506e3b5970c0514ef031a051dcd89100b73ad9d5e95a37e
+  data.tar.gz: bcb17191e3464ac5e3cce8071f0d3cadbe6985a9c17ea5bbffc1abd21f9cf5a6
 SHA512:
-  metadata.gz: 9a5499840827444cea022ef95ca205f52131084938a6417441eb73898961b726b02229d6f2e14964a77bf4fa24a537ae7b5c582fec1d7bcaf72f556e789dbd23
-  data.tar.gz: 6f53cf3ee9a63cd4906f2f09ede862f31440e573ea0d7033ce37c50d0d661d733dc7cb355e2a137e94f6a3eeb5e91904d2c37c7059039d145722a0b976b656cd
+  metadata.gz: bbd6c58beb6c449cc0a1947fa51f4f933e5dcac95b22369d0e3640f09d07190c3f925063ea428b044ab31e839fb45d6ca5c2c09a25478a8db1498fa9c176e57c
+  data.tar.gz: 94436933d34ff613b725d15e7686a87bf0c14b80a7b95800350df596075e356656b2bada4af3b35e79ccc704f54b247350db064eb98bc4d105a75484f30d43ec

data/lib/imogen/iiif/size.rb

@@ -9,8 +9,12 @@ class Size < Transform
       w = md[1] ? min(Integer(md[1]), @width) : nil
       h = md[2] ? min(Integer(md[2]), @height) : nil
       raise BadRequest.new("bad scale #{scale}") unless w or h
-      w ||= (@width * (h.to_f / @height)).round
-      h ||= (@height * (w.to_f / @width)).round
+      w ||= (@width * (h.to_f / @height))
+      h ||= (@height * (w.to_f / @width))
+      w = 1 if w > 0 && w < 1 # Avoid rounding very small numbers to zero
+      h = 1 if h > 0 && h < 1 # Avoid rounding very small numbers to zero
+      w = w.round
+      h = h.round
       e = [w,h]
     elsif md = /^pct:(\d+(\.\d+)?)/.match(scale)
       p = Float(md[1])
@@ -42,4 +46,4 @@ class Size < Transform
   end
 end
 end
-end
+end

data/lib/imogen.rb

@@ -6,7 +6,7 @@ module Imogen
   extend FFI::Library
 
   # Note: library_name function comes from `require 'vips'`
-  ffi_lib library_name("vips", 42)
+  ffi_lib FFI.library_name("vips", 42)
 
   attach_function :vips_cache_get_max_mem, [], :int
   attach_function :vips_cache_set_max_mem, [:int], :void

data/spec/unit/imogen_iiif_size_spec.rb

@@ -1,10 +1,11 @@
 require 'imogen/iiif'
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 describe Imogen::Iiif::Size, type: :unit do
-  before(:all) do
-    @test_image = ImageStub.new(175,131)
-  end
-  subject {Imogen::Iiif::Size.new(@test_image)}
+  let(:test_image_width) { 175 }
+  let(:test_image_height) { 131 }
+  let(:test_image) { ImageStub.new(test_image_width, test_image_height) }
+  subject {Imogen::Iiif::Size.new(test_image)}
+
   describe "#get" do
     describe "with scaling width" do
       it "should calculate for a good value" do
@@ -17,6 +18,14 @@ describe Imogen::Iiif::Size, type: :unit do
         expect{subject.get("0,")}.to raise_error Imogen::Iiif::BadRequest
         expect{subject.get("-2,")}.to raise_error Imogen::Iiif::BadRequest
       end
+
+      context "when the scaled width is a near-zero number before rounding is applied" do
+        let(:test_image_width) { 4096 }
+        let(:test_image_height) { 3 }
+        it "should round very small values to 1 (not 0)" do
+          expect(subject.get("512,")).to eql([512, 1])
+        end
+      end
     end
     describe "with scaling height" do
       it "should calculate for a good value" do
@@ -30,6 +39,14 @@ describe Imogen::Iiif::Size, type: :unit do
         expect{subject.get(",0")}.to raise_error Imogen::Iiif::BadRequest
         expect{subject.get(",-2")}.to raise_error Imogen::Iiif::BadRequest
       end
+
+      context "when the scaled height is a near-zero number before rounding is applied" do
+        let(:test_image_width) { 3 }
+        let(:test_image_height) { 4096 }
+        it "should round very small values to 1 (not 0)" do
+          expect(subject.get(",512")).to eql([1, 512])
+        end
+      end
     end
     describe "with scaling width and height" do
       it "should calculate for a good value" do
@@ -62,4 +79,4 @@ describe Imogen::Iiif::Size, type: :unit do
       end
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: imogen
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Ben Armintor
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-07 00:00:00.000000000 Z
+date: 2026-05-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-vips
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.3.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement