Skip to content

Conversation

@Irynei
Copy link
Owner

@Irynei Irynei commented Oct 4, 2018

  • Add DenseNet121 config.
  • Add test process.
  • Add early stopping.
  • Refactor.
return out


class DenseNet(nn.Module):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we could use a model from torchvision

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not for Cifar10. Models from torchvision are pre-trained on ImageNet and hence design to work with 224x224 input. In fact, it is stated in their documentation that images are expected to be at least of size 224x224.
I tried densenet121 from torchvision with Cifar10 and it didn't work.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use a constructor (e.g. torchvision.models.densenet.DenseNet), not a particular implementation.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is still some issue with AvgPool2d. (e.g. pytorch/vision#155, https://discuss.pytorch.org/t/torchvision-resnet-input-size/9405/2)
I will investigate how it could be done efficiently.

@Irynei Irynei merged commit 7f6bbc4 into master Oct 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants