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

Multi-pass processing #135

Draft
wants to merge 5 commits into
base: experimental
Choose a base branch
from
Draft

Multi-pass processing #135

wants to merge 5 commits into from

Conversation

BenBE
Copy link
Collaborator

@BenBE BenBE commented Jan 24, 2022

This branch will introduce multi-pass processing of the input image for cases when the internal face detection model cannot handle the full input image in one go due to the aspect ratio of the model compared to the aspect ratio of the input image.

This PR also contains a major cleanup of the code base by making the code style more consistent and splitting things into multiple files for more digestible chunks of code.

@BenBE BenBE added enhancement New feature or request experimental Things we want to play with outside the stable build labels Jan 24, 2022
@BenBE BenBE changed the base branch from main to experimental January 24, 2022 06:56
@BenBE BenBE force-pushed the multipass branch 2 times, most recently from e886c55 to b363e1f Compare February 1, 2022 18:31
app/deepseg.cc Show resolved Hide resolved
@phlash
Copy link
Collaborator

phlash commented Mar 4, 2022

Do we want to consider parallel processing the multiple passes? IMO this is probably not worth the extra pain unless we run very low on time budget..

@BenBE
Copy link
Collaborator Author

BenBE commented Mar 4, 2022

Do we want to consider parallel processing the multiple passes? IMO this is probably not worth the extra pain unless we run very low on time budget..

I'll do this as one process in the first version. We can still upgrade to multi-threading this processing in a second update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request experimental Things we want to play with outside the stable build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants