search

akamaster / pytorch_resnet_cifar10

Proper implementation of ResNet-s for CIFAR10/100 in pytorch that matches description of the original paper.
BSD 2-Clause "Simplified" License
1.22k stars 335 forks source link

an issue about resnet implementation #24

Closed xiaohua-chen closed 3 years ago

xiaohua-chen commented 4 years ago

Dear professor,I have two questions. 1、Why it is line 81: out += self.shortcut(x) but not out += x

2、For ImageNet/iNaturalist ResNet paper uses option 'B'.? For cifar10/cifar100 uses option 'A'. Not sure if I am understanding this right.

Look forward to your kind advic.

akamaster commented 3 years ago
  1. Please see #23 and comments within.
  2. Your understanding is correct. For ImageNet resnets, the configuration B was used historically. However for CIFAR10, the config A was used in the original paper.