Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HEIC.width and HEIC.height wrong #560

Closed
davidekholm opened this issue Dec 10, 2021 · 2 comments
Closed

HEIC.width and HEIC.height wrong #560

davidekholm opened this issue Dec 10, 2021 · 2 comments

Comments

@davidekholm
Copy link
Contributor

When inspecting the HEIC directory for HEIC images, the "width" and "height" tags are 512 and 512 pixels respectively. Shouldn't they reflect the image width and height?

I put some debug data inside ImageSpatialExtentsProperty.java and noticed that it's instantiated 3 times when reading metadata for an image. The 2:nd time with the correct values. However, they never make it to the final image.

Sample image: https://jalbum.net/download/IMG_9721.HEIC

@drewnoakes
Copy link
Owner

Hi David.

Are you able to provide that image to the test data library here so we can use it for regression testing? We don't have much coverage of HEIF/HEIC files.

I don't have time to dig into the details here at the moment. @payton added the HEIF support and has more context on this issue.

@payton
Copy link
Collaborator

payton commented Jan 30, 2022

Hi @davidekholm - This is a known bug. At the moment, we simply return the first image found (which may be a thumbnail or other auxiliary image) while we should be finding the primary image (as you pointed out).

Duplicate of #373

#445 was one PR put up in an attempt to resolve this issue. However, it still needs to be investigated (as per the PR comments).

@payton payton closed this as completed Jan 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants