Skip to content

Implement vips_tiffsave#1

Open
janhoy wants to merge 8 commits intocodecitizen:masterfrom
cominvent:vips_tiffsave
Open

Implement vips_tiffsave#1
janhoy wants to merge 8 commits intocodecitizen:masterfrom
cominvent:vips_tiffsave

Conversation

@janhoy
Copy link

@janhoy janhoy commented Nov 24, 2020

This PR adds:

  • Ability to save as TIFF (vips_tiffsave()), including the Pyramid TIFF format. All options from libvips are supported.
  • Configure libvips and glibc library locations through ENV variables JLIBVIPS_LIB_PATH and JLIBVIPS_GLIBC_PATH
  • Bumps versions of jna, groovy and gradle.
  • Updates README, with current release version and a new example
  • Fixed two test that crash on my Mac

I have tested these options from my own application: tile, tileHeight, tileWidth, compression (JPEG), quality and pyramid. Other options could still have bugs.

Have not added any new unit tests as I'm unfamiliar with the test framework.

I find it strange that running the normal build requires PGP passphrase. I'd like to report it as an issue, but the issue tracker is not enabled on this project :(

Upgrade jna, groovy and gradle
case 180: return D180;
case 270: return D270;
default:
throw new IllegalArgumentException("Allowed VipsAngle's are [0°, 90°, 180°, 270°].");
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was some ASCII encoding error for these characters on my mac, so I removed them

then:
tinted != null
println tinted
def dest = Paths.get("/Volumes/HD/backups/images/${colorName}.png")
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test failed due to hardcoded file path

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant