[PR #13377] [MERGED] fix(server): wrong image dimensions for RAW files (RAF, CR2) (also fixes face preview) #13933

Closed
opened 2026-02-05 15:23:33 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/13377
Author: @C-Otto
Created: 10/11/2024
Status: Merged
Merged: 11/1/2024
Merged by: @mertalev

Base: mainHead: raw-dimensions


📝 Commits (1)

📊 Changes

3 files changed (+90 additions, -3 deletions)

View changed files

📝 e2e/src/api/specs/asset.e2e-spec.ts (+72 -0)
📝 e2e/test-assets (+1 -1)
📝 server/src/services/metadata.service.ts (+17 -2)

📄 Description

For some RAW formats/cameras, the ImageWidth/ImageHeight information represents the dimensions of the preview image, i.e. much smaller than the actual image. This also causes issues with (face/people) thumbnail generation. The information in ImageSize seems to be correct, so we use this instead.

fixes #13049


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/13377 **Author:** [@C-Otto](https://github.com/C-Otto) **Created:** 10/11/2024 **Status:** ✅ Merged **Merged:** 11/1/2024 **Merged by:** [@mertalev](https://github.com/mertalev) **Base:** `main` ← **Head:** `raw-dimensions` --- ### 📝 Commits (1) - [`2737aab`](https://github.com/immich-app/immich/commit/2737aab055fc78ec964d4e1d56a2d47d645138b1) Add Fujifilm X100v RAW File (#12) ### 📊 Changes **3 files changed** (+90 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `e2e/src/api/specs/asset.e2e-spec.ts` (+72 -0) 📝 `e2e/test-assets` (+1 -1) 📝 `server/src/services/metadata.service.ts` (+17 -2) </details> ### 📄 Description For some RAW formats/cameras, the ImageWidth/ImageHeight information represents the dimensions of the preview image, i.e. much smaller than the actual image. This also causes issues with (face/people) thumbnail generation. The information in ImageSize seems to be correct, so we use this instead. fixes #13049 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 15:23:33 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#13933