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

problem of the model. #8

Open
ooobsidian opened this issue Apr 1, 2021 · 0 comments
Open

problem of the model. #8

ooobsidian opened this issue Apr 1, 2021 · 0 comments

Comments

@ooobsidian
Copy link

@seongmin-kye Hello, The full connection layer in the model is not defined at initialization time:

x = self.fc(x)

In Youngmoon Jung's repo, I noticed that in similar model code, he defined self.fc, and I guess you and Youngmoon Jung are from the same lab, so did you forget to define it here?

https://github.com/jymsuper/SpeakerRecognition_tutorial/blob/6dce64657632660a791e25624e4257c53c7257d6/model/resnet.py#L117

And in this case, the self.weight defined in the initialization is not used in the forward, where includes num_classes.

self.weight = Parameter(torch.Tensor(num_classes, 256))

So I could use any number of n_classes in testing and have poor performance:

n_classes = 5994 if args.data_type == 'vox2' else 1211

Is there anything wrong with what I mentioned?
Looking forward to your reply.

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

1 participant