Skip to content
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

VGG16BN and VGG19BN are not defined correctly #2

Open
alexlyzhov opened this issue Jul 31, 2019 · 0 comments
Open

VGG16BN and VGG19BN are not defined correctly #2

alexlyzhov opened this issue Jul 31, 2019 · 0 comments

Comments

@alexlyzhov
Copy link

VGGBase class has a loop for layer, activation in zip(layers, activations) which iterates over pairs of sequences of uneven length for VGGs with batch normalization. This causes blocks like Conv->BN->ReLU->Conv->BN->ReLU to be replaced by blocks like Conv->ReLU->BN->ReLU. It would probably be a good idea to use VGG16BN/VGG19BN definition from SWA/SWAG - it appears to be correct.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alexlyzhov