immich-app/immich
1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2026-02-05 00:30:57 +03:00
Code Issues 14 Packages Projects Releases 10 Wiki Activity

[BUG] Vertical videos have a too high resolution and have an inconsistent / to high bitrate #633

New Issue
Closed
opened 2026-02-04 21:35:05 +03:00 by OVERLORD · 4 comments
OVERLORD commented 2026-02-04 21:35:05 +03:00
Owner
Copy Link

Originally created by @akoyaxd on GitHub (Jan 28, 2023).

Describe the bug
If videos with vertical resolution get uploaded, the width is still 1280 so the hight is 2276 which is a way to high resolution for a thumbnail, often actually higher then the original Videofile.

To Reproduce
Steps to reproduce the behavior:

  1. Upload a video with vertical orientation
  2. wait til the preview is processed.
  3. download the preview from the browser and check the resolution

Expected behavior
The previews resolution has a max width and hight of 1280. So either 1280x720 or 720x1280.

Screenshots
grafik

System

  • Server Version: 1.43.1

Additional context
I recognized it after rerunning the transcoding job for all my video assets after the last update. Super awesome to have previews for my android Videos now, btw. Thanks for your hard work! :)

Originally created by @akoyaxd on GitHub (Jan 28, 2023). <!-- Note: Please search to see if an issue already exists for the bug you encountered. --> **Describe the bug** If videos with vertical resolution get uploaded, the width is still 1280 so the hight is 2276 which is a way to high resolution for a thumbnail, often actually higher then the original Videofile. **To Reproduce** Steps to reproduce the behavior: 1. Upload a video with vertical orientation 2. wait til the preview is processed. 3. download the preview from the browser and check the resolution **Expected behavior** The previews resolution has a max width and hight of 1280. So either 1280x720 or 720x1280. **Screenshots** <img width="1263" alt="grafik" src="https://user-images.githubusercontent.com/22196236/215282124-87600891-8682-4853-9a19-02a4e843f7f0.png"> **System** - Server Version: `1.43.1` **Additional context** I recognized it after rerunning the transcoding job for all my video assets after the last update. Super awesome to have previews for my android Videos now, btw. Thanks for your hard work! :)
OVERLORD commented 2026-02-04 21:35:10 +03:00
Author
Owner
Copy Link

@alextran1502 commented on GitHub (Jan 28, 2023):

@zackpollard Just FYI in case you want to add more options to the transcoding improvement.

@alextran1502 commented on GitHub (Jan 28, 2023): @zackpollard Just FYI in case you want to add more options to the transcoding improvement.
OVERLORD commented 2026-02-04 21:35:18 +03:00
Author
Owner
Copy Link

@bo0tzz commented on GitHub (Jan 29, 2023):

Potentially related to #795?

@bo0tzz commented on GitHub (Jan 29, 2023): Potentially related to #795?
OVERLORD commented 2026-02-04 21:35:21 +03:00
Author
Owner
Copy Link

@akoyaxd commented on GitHub (Jan 29, 2023):

I set [vf scale=]1280:1280:force_original_aspect_ratio=decrease form the mentioned issue. Thank to @raisinbear for sharing your research results.

The previews do now transcode to the correct resolution. They are still a bit large for a preview. While the crf Setting is nice, I think a setting for the bitrate would be make sense here as well, to make the playback of previews more consistent in loading times.

@akoyaxd commented on GitHub (Jan 29, 2023): I set [vf scale=]`1280:1280:force_original_aspect_ratio=decrease` form the mentioned issue. Thank to @raisinbear for sharing your research results. The previews do now transcode to the correct resolution. They are still a bit large for a preview. While the crf Setting is nice, I think a setting for the bitrate would be make sense here as well, to make the playback of previews more consistent in loading times.
OVERLORD commented 2026-02-04 21:35:27 +03:00
Author
Owner
Copy Link

@zackpollard commented on GitHub (Jan 30, 2023):

We are planning to further improve the transcoder options. I'll be doing some work over the next week, we are going to simplify the settings a little so we can further improve on the transcoder and how it handles the different scenarios. The scaling filter will be going for example, and instead we will be showing an option like Resolution: 720p/1080p/1440p/4k in a dropdown. The server will then handle all the specific cases such as vertical videos etc

@zackpollard commented on GitHub (Jan 30, 2023): We are planning to further improve the transcoder options. I'll be doing some work over the next week, we are going to simplify the settings a little so we can further improve on the transcoder and how it handles the different scenarios. The scaling filter will be going for example, and instead we will be showing an option like Resolution: 720p/1080p/1440p/4k in a dropdown. The server will then handle all the specific cases such as vertical videos etc
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
accessibility
changelog:enhancement
changelog:security
changelog:translation
cli
date-time
documentation
external-library
format
good first issue
needs-answer
nice to have
sharing
tech-debt
📱mobile
🖥️web
🗄️server
🧠machine-learning
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#633
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?