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

Add configurable maximum transcode sizes #178

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

WithoutPants
Copy link
Collaborator

Adds options to the configuration page to allow setting a maximum size for generated transcoded files, and for streaming transcoded files.

Default value is "Original" which uses the existing resolution of the source video. Other values are "240p", "480p", "720p", "1080p", "4K". The transcoding code uses this as the maximum size to down-scale to. If a scene has a lower resolution than the max, then that resolution is used. This scaling is applied to the height for normal landscape videos, and applied to the width for portrait videos.

Should help address the performance issues identified in #31. Related to #10.

@bnkai
Copy link
Collaborator

bnkai commented Nov 4, 2019

Tested for transcode and livestream sizes , everything seems ok.
Now even 1080p live transcoded files can be seen without freezes.

@Leopere Leopere added this to In progress in Wants via automation Nov 4, 2019
@Leopere Leopere added this to In progress in Version 0.2.0 via automation Nov 4, 2019
@Leopere Leopere added the improvement Something needed tweaking. label Nov 4, 2019
@Leopere Leopere merged commit d1ea2ff into stashapp:master Nov 4, 2019
Wants automation moved this from In progress to Done Nov 4, 2019
Version 0.2.0 automation moved this from In progress to Done Nov 4, 2019
@WithoutPants WithoutPants deleted the configure_transcode_size branch May 15, 2020 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Something needed tweaking.
Projects
No open projects
Version 0.2.0
  
Done
Wants
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants