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

Return face bounding boxes along landmarks #227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alkoumpa
Copy link

@alkoumpa alkoumpa commented Dec 2, 2020

Hi,

These changes allow the user to also return the face bounding box, when calling get_landmarks_from_image or get_landmarks_from_batch, I think it doesn't break existent code and enables a second useful output.

The option to enable this feature is passed as the last argument of FaceAlignment constructor (return_face_results). By having it disabled by default, the return values of the two get_landmarks_* remain as they currently are (only landmarks), therefore doesn't break existent code that expects single output. Also because of the default value in the constructor, it doesn't object creation with less or named arguments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant