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

Input to SyncNet #2

Open
DomhnallBoyle opened this issue Apr 29, 2021 · 1 comment
Open

Input to SyncNet #2

DomhnallBoyle opened this issue Apr 29, 2021 · 1 comment

Comments

@DomhnallBoyle
Copy link

Hi, I'm a bit confused about different implementations.

The SyncNet paper suggests that the image input to the SyncNet model is of mouth images and you describe the procedure to extract the mouth region here in this issue: #1 (comment)

Although, in the official repo the author mentions that the input to the pre-trained model is actually the full face and not the mouth region. They don't implement the cropping of the mouth region in their code: joonson/syncnet_python#9

So is the pretrained SyncNet v4/v7 models for frontal and multi-view on the website here actually on the entire face? Wouldn't your code give inaccurate results for the model outputs since you use the mouth region instead of the full face?

Thanks

@voletiv
Copy link
Owner

voletiv commented May 4, 2021

Hi,

I believe joonsoon's code was released after the paper, so perhaps some details are different. The Syncnet weights that I had worked for the LRW dataset, but joonsoon's code is faster and more accurate, it would be better to use that.

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

2 participants