[Bug fix] redundant layers in ResNet #141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

yunjey merged 1 commit into yunjey:master from keineahnung2345:fix-resnet

Nov 6, 2018

Contributor

keineahnung2345 commented Nov 6, 2018

In https://github.com/yunjey/pytorch-tutorial/blob/master/tutorials/02-intermediate/deep_residual_network/main.py#L115, it defined a length 4 layers.
But in https://github.com/yunjey/pytorch-tutorial/blob/master/tutorials/02-intermediate/deep_residual_network/main.py#L84, it only uses layers[0] and layers[1].
So the last entry of [2,2,2,2] should be redundant.


          [Bug fix] redundant layers in ResNet

606d0aa

In https://github.com/yunjey/pytorch-tutorial/blob/master/tutorials/02-intermediate/deep_residual_network/main.py#L115, it defined a length 4 `layers`. 
But in https://github.com/yunjey/pytorch-tutorial/blob/master/tutorials/02-intermediate/deep_residual_network/main.py#L84, it only uses `layers[0]` and `layers[1]`. 
So the last entry of [2,2,2,2] should be redundant.

Owner

yunjey commented Nov 6, 2018

@keineahnung2345 You are right. Thanks very much.

yunjey merged commit c48008b into yunjey:master

keineahnung2345 deleted the fix-resnet branch

April 30, 2020 12:42

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

Labels

None yet