ImageSharp icon indicating copy to clipboard operation
ImageSharp copied to clipboard

ExifTag.GPSLatitudeRef gets resolved to ExifTagValue.InteroperabilityIndex

Open eangelov opened this issue 3 years ago • 6 comments

Prerequisites

  • [X] I have written a descriptive issue title
  • [X] I have verified that I am running the latest version of ImageSharp
  • [X] I have verified if the problem exist in both DEBUG and RELEASE mode
  • [X] I have searched open and closed issues to ensure it has not already been reported

Description

ExifTag.GPSLatitudeRef when converted to string is resolved as "InteroperabilityIndex" image image

Steps to Reproduce

using SixLabors.ImageSharp.Metadata.Profiles.Exif;
Console.WriteLine( ExifTag.GPSLatitudeRef );

System Configuration

  • Packages: image

  • Environment (Operating system, version and so on): image

  • .NET Framework version: .Net 6.0

eangelov avatar Feb 11 '22 14:02 eangelov

Both ExifTagValue.InteroperabilityIndex and ExifTagValue.GPSLatitudeRef have the same value and it looks like the first value is displayed when the metadata is displayed so this is not an issue. I do wonder if we should remove the ExifTagValue.InteroperabilityIndex value @JimBobSquarePants.

dlemstra avatar Feb 12 '22 17:02 dlemstra

After doing a quick search in Magick.NET it looks like InteroperabilityIndex is not included in ExifTagValue, removing it looks reasonable.

eangelov avatar Feb 14 '22 07:02 eangelov

Removing the value prevents us ever supporting the tag. ExifTagValue should actually be 3 different enums.

https://www.awaresystems.be/imaging/tiff/tifftags/privateifd/interoperability.html

JimBobSquarePants avatar Mar 08 '22 03:03 JimBobSquarePants

I came here to make a similar report, and found this issue already open. @dlemstra's comment is not true for me: ExifTagValue.InteroperabilityIndex has the value that I would expect to find in ExifTagValue.GPSLatitudeRef, and ExifTagValue.GPSLatitudeRef has no value at all.

bjh0 avatar Nov 06 '22 16:11 bjh0

We should split up the enum for V3.

JimBobSquarePants avatar Nov 07 '22 01:11 JimBobSquarePants

This never made V3 but we should target V4 for this.

JimBobSquarePants avatar Sep 27 '23 01:09 JimBobSquarePants