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

Option to extract a number of video frames as a function of video duration #1107

Closed
lfcnassif opened this issue May 12, 2022 · 3 comments · Fixed by #1473
Closed

Option to extract a number of video frames as a function of video duration #1107

lfcnassif opened this issue May 12, 2022 · 3 comments · Fixed by #1473
Assignees
Projects

Comments

@lfcnassif
Copy link
Member

As discussed in other issues #1081 #1102, this could improve search results of a number of video tasks (nudity and faces detection, future similarity search, OCR and photoDNA and others).

extractFramesMillisecondInterval = 1000 would be a good default value for pedo and forensic profiles and disabled for others?

Or should we make some kind of attenuation targeting huge videos, maybe using a logarithmic or similar function applied to video duration?

@lfcnassif lfcnassif changed the title Option to extract a number of video frames proportional to video duration Option to extract a number of video frames as a function of video duration May 12, 2022
@lfcnassif
Copy link
Member Author

I'm increasing the priority of this, this can be useful to #1472.

@lfcnassif lfcnassif self-assigned this Jan 12, 2023
@lfcnassif
Copy link
Member Author

@tc-wleite I'm coding to change just the number of subitems extracted (if enabled) to don't mess the mosaic already generated, ok?

@wladimirleite
Copy link
Member

@tc-wleite I'm coding to change just the number of subitems extracted (if enabled) to don't mess the mosaic already generated, ok?

Sure! I think it should be fine, as the algorithm that generates the mosaic can receive more images than it actually uses.
I will be out next week, but I can take a look in the changes and test if you want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
4.1
Done
Development

Successfully merging a pull request may close this issue.

2 participants