PreResNet - wrong implementation of shortcut

facebookresearch / mixup-cifar10

mixup: Beyond Empirical Risk Minimization

Other

1.16k stars 225 forks source link

Open emrahyigit opened 3 years ago

emrahyigit commented 3 years ago

Hello everyone,

I have two questions.

Why the repository name is cifar10. Is it because mixup is solely designed for CIFAR-10?
And in the repo, doesn't matter if there is a shortcut or not, still, BN-Relu applied to the input. This is different in many other repositories. I just want to learn which one is correct?

rederyang commented 3 years ago

The 1st question, mixup is not only for cifar10. Named cifar10 because they only provide the experiment codes for cifar 10. Please check the paper.