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

Handle the case when the video contains frames with no face #10

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

jhj0517
Copy link
Owner

@jhj0517 jhj0517 commented Jan 14, 2025

Related issues / PRs.

Summarize Changes

  1. Handle the exception case that there might be frames with no face. Even if there are no faces at all in the video, it still needs to go through the pipeline because pipeline resize the frames of the video.
    If the video contains both frames with faces and no faces, one side has to be resized to other side, because the end result should be a video where all frames are the same size.
    So just pass to the pipeline to resize the frames even if there's no face at all.

@jhj0517 jhj0517 added the bug Something isn't working label Jan 14, 2025
@jhj0517 jhj0517 changed the title Handle frame with no face Handle the case when the video contains frames with no face Jan 14, 2025
@jhj0517 jhj0517 merged commit 5051132 into master Jan 14, 2025
@jhj0517 jhj0517 deleted the fix/no-face branch January 14, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant