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

Would it be possible to enable nvenc in ffmpeg when classifying videos? #655

Open
relink2013 opened this issue Jan 27, 2023 · 3 comments
Open
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@relink2013
Copy link

Describe the feature you'd like to request

When classifying videos Recognize pegs all CPU cores to 100% with an ffmpeg process. I assume this is for the video tagging. If its possible to add the option to enable nvenc for the ffmpeg process it would help tremendously.

Describe the solution you'd like

Ultimately it would be great to see recognize ship with the nvenc enabled build of ffmpeg.

But for starters I'd be thrilled to just have a config.php setting I can add/change to enable nvenc, and point recognize to my static build I already have with nvenc enabled.

Describe alternatives you've considered

None so far, I only just realized this about five minutes ago.

@relink2013 relink2013 added the enhancement New feature or request label Jan 27, 2023
@marcelklehr
Copy link
Member

Recognize pegs all CPU cores to 100% with an ffmpeg process

I'll try to see if I can make the ffmpeg process respect the cores limit set in the settings.

I'm not sure if nvenc would solve this issue. Possibly we'd need nvdec
, but that only supports limited file types.

@relink2013
Copy link
Author

Ah, that would make sense that you only really need to decode the video. But nvdec appears to support all the major file types.

Maybe just make it optional?

marcelklehr added a commit that referenced this issue Feb 22, 2023
@marcelklehr marcelklehr added the good first issue Good for newcomers label Jun 5, 2023
@marcelklehr
Copy link
Member

We could add an option in the admin settings to add additional command line arguments to ffmpeg

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants