Coder Social home page Coder Social logo

Comments (26)

rudi123github avatar rudi123github commented on August 23, 2024

Ok, I figured out that the change needs to happen in pkg/ffmpeg/encoder_scene_preview_chunk.go. I added the same parameter I found in another module as extra argument to function ScenePreviewVideoChunk:

"-max_muxing_queue_size", "1024", // https://trac.ffmpeg.org/ticket/6375

I rebuilt stash with the docker container and now everything runs fine locally. I cannot push the update to github, so I ask one of the devs to make this change please.

from stash.

Leopere avatar Leopere commented on August 23, 2024

@rudi123github did you still get this issue or can it be closed?

from stash.

rudi123github avatar rudi123github commented on August 23, 2024

I fixed this in my local build but the change should go into the development branch.

from stash.

Leopere avatar Leopere commented on August 23, 2024

If you file a PR it can get reviewed / worked on. As far as I'm aware this app is far from 1.0 status but whatever works.

from stash.

rudi123github avatar rudi123github commented on August 23, 2024

Okay, I just figured out how to do that. The PR is there now.

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

I am still getting errors when trying to generate preview images.
Since my filename contains spaces, I am not sure if it should be quoted.

Can anyone confirm it works with spaces?
Also the -max_muxing_queue_size 1024 is missing.

msg="ffmpeg error when running command </usr/bin/ffmpeg -v quiet -ss 1562 -y -i /data/MyVideo [x264].mp4 -vframes 1 -q:v 1 -vf scale=160:-1 -f image2 /generated/tmp/thumbnail071.jpg>"

from stash.

rudi123github avatar rudi123github commented on August 23, 2024

Hello, this is a different function. Can you please run this from a terminal:

/usr/bin/ffmpeg -ss 1562 -y -i "/data/MyVideo [x264].mp4" -vframes 1 -q:v 1 -vf scale=160:-1 -f image2 /generated/tmp/thumbnail071.jpg

and post the output here?

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

If I encapsulate the filename with " it works, otherwise I get errors.

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

@Leopere : I pulled the latest docker image but I am still getting the error above? Could it be that my PR ist not included in the docker build?

from stash.

Leopere avatar Leopere commented on August 23, 2024

@ExceptionalError looks like everything I merged was built I wonder why its not working as prescribed. For good measure I've rebuilt it one more time just in case it didn't work properly the last time but I doubt that anything will change.

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

Something changed, there are now quotes but somehow they get escaped:

/usr/bin/ffmpeg -v quiet -ss 2320 -y -i \"/data/MyVideo [x264].mp4\" -vframes 1 -q:v 1 -vf scale=160:-1 -f image2 /generated/tmp/thumbnail080.jpg

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

I have tried some more fixes in #131 - what do you think?

from stash.

rudi123github avatar rudi123github commented on August 23, 2024

I wonder if this is really related to spaces. I have many files with spaces in their names and all of these work without problems (I am on Linux). I think it must be a different character. Does your fixes version work for you?

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

I think it must be a different character

Could you try renaming one of your files like this [MyTag] MyFilename (09.10.2019) [x264].mkv and see if you get errors, too?

from stash.

bnkai avatar bnkai commented on August 23, 2024

@ExceptionalError on linux at least spaces and your naming scheme works fine. The quoted part changes you did to 117 though broke that . #131 breaks the screenshot generation in the linux build also

from stash.

jpegsarebad avatar jpegsarebad commented on August 23, 2024

Having this error with the most recent linux build, is this a different issue or is this still unfixed?
ERRO[0354] ffmpeg error when running command </usr/bin/ffmpeg -v quiet -ss 49.612 -y -i /home/jack/Applications/stash/videos/vid.mp4 -vframes 1 -q:v 5 -vf scale=320:-1 -f image2 screenshots/650b216cccc90fdb9bfeeab412c6c06c.thumb.jpg>
ERRO[0354] ffmpeg error when running command </usr/bin/ffmpeg -v quiet -ss 49.612 -y -i /home/jack/Applications/stash/videos/vid.mp4 -vframes 1 -q:v 2 -vf scale=1280:-1 -f image2 screenshots/650b216cccc90fdb9bfeeab412c6c06c.jpg>

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

@bnkai : Thanks for the feedback. Any idea how to proceed?
When executing manually in Docker (with quotes) I can generate the previews successfully.
I changed #131 so it outputs the ffmpeg error message, too - maybe that's a start.

from stash.

bnkai avatar bnkai commented on August 23, 2024

@jpegsarebad you can try executing the command manually to see if that produces an error
e.g.

usr/bin/ffmpeg -v quiet -ss 49.612 -y -i /home/jack/Applications/stash/videos/vid.mp4 -vframes 1 -q:v 5 -vf scale=320:-1 -f image2 thumb.jpg

doing that at least we can get a hint if something's wrong with the video file or with the invocation of ffmpeg through stash

@ExceptionalError is the error for all the files or just some specific? you say you used docker, if you use the plain linux binary then instead of docker do you get the same errors? i haven't used the docker but as i said the linux binary doesn't have any problem with spaces or special characters like ( [ + in the filename

from stash.

jpegsarebad avatar jpegsarebad commented on August 23, 2024

@bnkai Yes, running the command manually works with no errors and produces the image correctly.

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

@bnkai I get the error message for all my files. When executing the command in the docker image manually, it runs successfully.
Imho #131 needs to be merged so there is at least the error message in the log.

from stash.

jpegsarebad avatar jpegsarebad commented on August 23, 2024

All of a sudden it started working, having changed nothing as far as I can remember. But the videos that had already been entered without sprites did not have them, even after running 'generate.' To solve this, I deleted the database and started anew. When I added my videos back in, the error came back.

Now, when using 'generate' I get the following error:
INFO[0941] [generator] generating sprite image for /videos/Nia Nacci and Honey Gold - Do I Know You?.mp4
ERRO[0952] error generating sprite: open vtt/f1f8cc3153dea5f75ca2582cb4d1bf51_sprite.jpg: no such file or directory

EDIT: working again, but only vtts and previews, not any sprites.

from stash.

Leopere avatar Leopere commented on August 23, 2024

@ExceptionalError #131 is just waiting on confirmation from @WithoutPants and @bnkai to verify that there's nothing broken in the PR.

from stash.

bnkai avatar bnkai commented on August 23, 2024

@ExceptionalError if the error is for all the files have you checked that the folders you use for the metadata files have the right permissions or access rights ?
For #131 a simple extra print doesn't break anything but have you tried that ? i don't have an easy way to force an error just to check that....

from stash.

bnkai avatar bnkai commented on August 23, 2024

@jpegsarebad the only think i can think off is also maybe there might be someting wrong with the permissions. have you checked that all the metadata directories are ok?
afaik the sprite images are created in the tmp directory and then concatenated to a single image file in the meta vtt folder

from stash.

ExceptionalError avatar ExceptionalError commented on August 23, 2024

Whilst the log output it still not in the current docker build, I mounted the "/generated" folder with the docker compose file and everything seems to be working fine now.
Maybe the folder structure inside the container is not properly created?

from stash.

Leopere avatar Leopere commented on August 23, 2024

Since we merged that PR I'm going to close this but feel free to continue the conversation or start a new issue if theres any further discussion to be had.

from stash.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.