This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

PreResNet - wrong implementation of shortcut #20

Open

emrahyigit opened this issue Sep 28, 2020 · 1 comment

emrahyigit commented Sep 28, 2020

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?

mixup-cifar10/models/resnet.py

Line 64 in eaff31a

def forward(self, x):

rederyang commented Apr 27, 2021 •

edited

Loading

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.