Skip to content

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
@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