Skip to content

Comments

Layers in order#602

Merged
jeffdonahue merged 3 commits intoBVLC:devfrom
kloudkl:layers-in-order
Jul 4, 2014
Merged

Layers in order#602
jeffdonahue merged 3 commits intoBVLC:devfrom
kloudkl:layers-in-order

Conversation

@kloudkl
Copy link
Contributor

@kloudkl kloudkl commented Jul 3, 2014

In reponse to @shelhamer's advice, ArgmaxLayer, ConcatLayer, FlattenLayer, SoftmaxLayer and SplitLayer are moved into common_layers. ConvolutionLayer, EltwiseLayer, Im2colLayer, InnerProductLayer, LRNLayer and PoolingLayer stay in the vision_layers. But your mileage may vary.

The data layers and the loss layers are in alphabetical order with the exception that the LossLayer is above its subclasses.

@shelhamer
Copy link
Member

This looks neatly arranged to me. @jeffdonahue please take a look and merge if you like this organization.

jeffdonahue added a commit that referenced this pull request Jul 4, 2014
@jeffdonahue jeffdonahue merged commit eb40cb4 into BVLC:dev Jul 4, 2014
@jeffdonahue
Copy link
Contributor

Looks good to me. I think InnerProductLayer and EltwiseLayer could also be moved to common_layers -- generally I'd say any layer that doesn't care about the height/width structure of the input seems like it shouldn't be specifically called a "vision layer".

@kloudkl kloudkl deleted the layers-in-order branch July 4, 2014 06:10
mitmul pushed a commit to mitmul/caffe that referenced this pull request Sep 30, 2014
RazvanRanca pushed a commit to RazvanRanca/caffe that referenced this pull request Nov 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants