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

Add face detection #524

Open
wants to merge 107 commits into
base: master
Choose a base branch
from
Open

Conversation

i-Still-Believe
Copy link

@i-Still-Believe i-Still-Believe commented Feb 20, 2020

@Fang-Haoshu
Copy link
Member

Thanks jiasong! Nice works.

Here are some modifications before merging.

  1. I found that there are some *.pyc files, could you please remove them and add *.pyc to the .gitignore
  2. The flag should be --face. And we don't need a write_face. You can append the face keypoints after the original keypoints, that means, we have 51+17 keypoints now. We do not need to render the face independently. You can move that part into vis_frame and vis_frame_fast
  3. we can move all the centerface model files to google drive
  4. we can rename centerface to face
  5. There are some useless files under centerface/utils. Please check them and remove all useless files and functions

@ZhengLeon
Copy link

Hi@ @7-GUO-7 ,
The face detection models you shared are unavailable now, could you please
upload again?

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.

3 participants