-
Notifications
You must be signed in to change notification settings - Fork 26
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
Ilo #74
base: main
Are you sure you want to change the base?
Conversation
|
||
|
||
def get_landmark(filepath): | ||
"""get landmark with dlib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docstrings consistent with other files
|
||
def align_face(filepath): | ||
""" | ||
:param filepath: str |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docstrings consistent with other files
lensless/utils/align_face.py
Outdated
|
||
lm = get_landmark(filepath) | ||
|
||
# lm_chin = lm[0:17] # left-right |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove commented code that isn't needed
Thank you for submitting a pull request (PR)! Before hitting the submit button, we would be really glad if you check all the following points. Even if you can't check all the boxes below, do not hesitate to go ahead with the PR and ask for help there.
Please also refer to the doc on contributing for more details.
pytest
from the repository root? See the documentation for more details.Happy PR 😃