Skip to content

Conversation

@thalitadru
Copy link

I implemented slim models for DenseNet and FC-DenseNet, based on the original lasagne/theano code made available by the authors of FC-DenseNets. Hope it will be useful.

@bhack
Copy link
Contributor

bhack commented Sep 15, 2017

/cc @ikhlestov

@bhack bhack mentioned this pull request Nov 19, 2017
@dongzhuoyao
Copy link

why closed?

@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

Copy link

@ikhlestov ikhlestov left a comment

Choose a reason for hiding this comment

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

Somehow I was mentioned in the review comments. I've checked the images DenseNet and it seems to me ok. However, It will be great if someone can use network with various input shapes, not depending on the last average pooling with 7x7 filters.

# We perform a first convolution.
# If DenseNet BC, first convolution has 2*growth_rate output channels
if bottleneck and compression < 1.0:
n_filters_first_conv = 2 * growth_rate

Choose a reason for hiding this comment

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

I think this behavior should be somehow be mentioned in the densenet method documentation.

#####################
pool_name = 'pool%d' % (n_dense + 1)
if dense_prediction:
net = slim.pool(net, [7, 7], pooling_type='AVG', scope=pool_name)

Choose a reason for hiding this comment

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

What if we have not 224x224 input? From my point of view we should use dynamic average pooling based on input shape.

from __future__ import division
from __future__ import print_function

import numpy as np

Choose a reason for hiding this comment

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

Just unused import

@dongzhuoyao
Copy link

@thalitadru

@saberkun saberkun closed this Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants