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

Why 112x112? #4

Open
yaseryacoob opened this issue Mar 17, 2024 · 5 comments
Open

Why 112x112? #4

yaseryacoob opened this issue Mar 17, 2024 · 5 comments

Comments

@yaseryacoob
Copy link

In reading the paper and the repo I am wondering why you chose 112x112 as the face resolution? Given other embeddings are in the 200+ and given that one could go to 512x512 to capture Identity even better.

thanks for sharing your code.

@mapooon
Copy link
Owner

mapooon commented Mar 17, 2024

Thank you for your interest in our work.
This is because the face recognition (identity encoding) part of our code is based on insightface that uses 112x112 images.
In practice, 112x112 images work well in recognizing (identifying) faces.

@yaseryacoob
Copy link
Author

The issue is that face recognition type embeddings are (or maybe) good for face recognition, but more is needed for generative type purposes. This is why you had to have the expression encoding. I wonder if a richer embedding can be computed. It is an open question, beyond your work. Thanks for sharing your work!

@mapooon
Copy link
Owner

mapooon commented Mar 17, 2024

Exactly. 112x112 images are considered sufficient for face recognition, but not obvious for generative purposes. Higher resolution may yield better results for generative models as you mentioned.

@gaoyixuan111
Copy link

Thank you for your excellent work. Could you please share your training process readme and training time,datasets?

@AndyG-BUPT
Copy link

Thank you for your excellent work. Could you please share your training process readme and training time,datasets?

请问您有得到相关代码吗

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

No branches or pull requests

4 participants