Skip to content

Add APIs to get image EXIF data and ICC profile data#317

Merged
tonimelisma merged 1 commit intodavidbyttow:masterfrom
AttilaTheFun:lshire-exif-and-icc
Oct 24, 2022
Merged

Add APIs to get image EXIF data and ICC profile data#317
tonimelisma merged 1 commit intodavidbyttow:masterfrom
AttilaTheFun:lshire-exif-and-icc

Conversation

@AttilaTheFun
Copy link
Contributor

These are two APIs I've wanted for a while and I finally got around to implementing them + making a pull request.

The first, GetICCProfile, is related to HasICCProfile but it actually returns the byte slice resulting from invoking vips_image_get_blob with the VIPS_META_ICC_NAME parameter.

This is useful for me as well as hopefully other consumers because I want to parse the data with LCMS2 and determine the ICC profile name and whether it's a wide color profile or not.

The second API, GetExif, is related to ImageFields but it actually returns the string values in a map for all of the image fields with an "exif" prefix.

This is useful for pulling interesting information out of images like the type of camera and lens.

@AttilaTheFun
Copy link
Contributor Author

@tonimelisma @davidbyttow Hey folks! I remember we talked about this a long time ago and I finally got around to implementing it. Let me know what you think!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 76.895% when pulling 6c23f97 on AttilaTheFun:lshire-exif-and-icc into 5f0601e on davidbyttow:master.

@tonimelisma tonimelisma enabled auto-merge (squash) October 24, 2022 02:28
Copy link
Collaborator

@tonimelisma tonimelisma left a comment

Choose a reason for hiding this comment

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

lgtm

@tonimelisma tonimelisma merged commit ce49ee9 into davidbyttow:master Oct 24, 2022
@AttilaTheFun
Copy link
Contributor Author

Awesome, thanks!

Elad-Laufer pushed a commit to wix-incubator/govips-contrib that referenced this pull request Nov 22, 2022
Elad-Laufer added a commit to wix-playground/govips that referenced this pull request Nov 22, 2022
* Original image type (davidbyttow#299) * - add golden * - freeCString abstraction * When implicitly converting the image type (BMP to PNG), keep the original format available for query * gaussian blur support min_ampl (davidbyttow#307) Co-authored-by: muyouran <muyouran@pinduoduo.com> * Add APIs to get image EXIF data and ICC profile data (davidbyttow#317) * Add support for high-bit-depth AVIF and HEIF images (davidbyttow#322) * Fix bug with icc transform bit depth (davidbyttow#323) * Determine libvips version runtime (davidbyttow#320) * add ExtractBandToImage (davidbyttow#324) * Update image.go Add function to extract a band into a new image instead of replacing the current image ref * Update image.go * add test Co-authored-by: Roffe <roffe@roffe.nu> * BandSplit() split an n-band image into n separate images (davidbyttow#326) * BandSplit() split an n-band image into n separate images * move tests Co-authored-by: Roffe <roffe@roffe.nu> * Bump CI to latest Ubuntu (davidbyttow#329) * Improved exif extraction (davidbyttow#325) * Add support for reading + writing EXIF data * Fix test for removing exif * Cleanup tests * Fix removes exif test on ubuntu * TIFF may not iptc data * Cleanup test images * add images for ubuntu 22 lts (davidbyttow#328) Co-authored-by: Roffe <roffe@roffe.nu> * don't use deprecated functions Co-authored-by: Qisen <1620671486@qq.com> Co-authored-by: muyouran <muyouran@pinduoduo.com> Co-authored-by: Logan Shire <logan.shire@gmail.com> Co-authored-by: tgmpje <65441725+tgmpje@users.noreply.github.com> Co-authored-by: Joakim Karlsson <5434736+roffe@users.noreply.github.com> Co-authored-by: Roffe <roffe@roffe.nu> Co-authored-by: Toni Melisma <59022391+tonimelisma@users.noreply.github.com>
@AttilaTheFun AttilaTheFun deleted the lshire-exif-and-icc branch April 24, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants