Confusezius/Deep-Metric-Learning-Baselines

question about code in "googlenet.py"

Closed this issue · 2 comments

kdwonn commented

Thanks for your code and recent researches.

The question is, it seems like code presented in https://github.com/Confusezius/Deep-Metric-Learning-Baselines/blob/master/googlenet.py is for Inception-BN, not googlenet. Am I missing something?

kdwonn commented

Got an answer from pytrorch issues. - https://github.com/pytorch/vision/issues/906
So "googlenet.py" is not the same as the original "googlenet" presented in "Going Deeper .. ". It has different kernel sizes and even uses BN.

I think I should reference Tensorflow/Pytorch googlenet code instead of "Going deeper .. " in my next paper.

Confusezius commented

Thanks for noticing! I just took the GoogLeNet from the official Torchvision Repo, since they also reference the original paper, but you are right. I will add a disclaimer for that :).