meta-SR icon indicating copy to clipboard operation
meta-SR copied to clipboard

problem of the model.

Open ooobsidian opened this issue 3 years ago • 0 comments

@seongmin-kye Hello, The full connection layer in the model is not defined at initialization time: https://github.com/seongmin-kye/meta-SR/blob/b4c1ea1728e33f7bbf7015c38f508f24594f3f88/model/resnet_256.py#L157

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. https://github.com/seongmin-kye/meta-SR/blob/b4c1ea1728e33f7bbf7015c38f508f24594f3f88/model/model.py#L31

So I could use any number of n_classes in testing and have poor performance: https://github.com/seongmin-kye/meta-SR/blob/b4c1ea1728e33f7bbf7015c38f508f24594f3f88/EER_full.py#L37

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

ooobsidian avatar Apr 01 '21 11:04 ooobsidian