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

Preview generation fallback #725

Merged

Conversation

JoeSmithStarkers
Copy link
Contributor

When a preview generation fails (often for wmv/avi files), the new code tries with less stricted (no xerror) and more time consuming options (slow+fast seek). Fixes #635

Fix a minor issue when stash downloads ffmpeg/ffprobe, but doesn't re-detect paths.

Added comment about restoring a backup with a move operation.
Added .DS_Store to ignore list (mac osx annoying file)
Added pkg/pkg to ignore list (can't seem to configure goland to use the vendor dir)

@JoeSmithStarkers JoeSmithStarkers force-pushed the feature/preview_fallback branch 2 times, most recently from 1a40385 to 85cc9e7 Compare August 12, 2020 04:17
@bnkai
Copy link
Collaborator

bnkai commented Aug 13, 2020

Seems to work ok.
Tried with a couple of avis that where throwing a header missing error.
As a sidenote setting the start time to say 2% seems to reduce the specific error cases ( i had that set before the test and the file would generate without error / fallback. Removed that and the fallback method was triggered and worked as intended ).

EDIT. Added to milestone for v0.3 as whis would save as a lot of user help requests.

@bnkai bnkai added this to the Version 0.3.0 milestone Aug 13, 2020
When a preview generation fails (often for wmv/avi files), the new code tries with less stricted (no xerror) and more time consuming options (slow+fast seek).
Fix a minor issue when stash downloads ffmpeg/ffprobe, but doesn't re-detect their paths.
@WithoutPants WithoutPants merged commit ecc42e4 into stashapp:develop Aug 16, 2020
Tweeticoats pushed a commit to Tweeticoats/stash that referenced this pull request Feb 1, 2021
* Added preview generation fallback feature.
When a preview generation fails (often for wmv/avi files), the new code tries with less stricted (no xerror) and more time consuming options (slow+fast seek).
Fix a minor issue when stash downloads ffmpeg/ffprobe, but doesn't re-detect their paths.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cannot generate .wmv sprites and markers
3 participants