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.

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

Split monolithic LayerParameter into per-layer Protobuf Messages #208

Closed
shelhamer opened this issue Mar 13, 2014 · 3 comments
Closed

Split monolithic LayerParameter into per-layer Protobuf Messages #208

shelhamer opened this issue Mar 13, 2014 · 3 comments
Assignees
Milestone

Comments

@shelhamer
Copy link
Member

Split LayerParameter into PoolingLayerParameter, ConvolutionLayerParameter, and so on for layer-specific params. This will be self-documenting in terms of what params each layer accepts.

There should perhaps be a "base" LayerParameter with common fields.

Layers and LayerConnections should could go away. There's no need for the

layers {
   layer {
     ..
   }
 }

incantation.

For backwards-compatibility, the current caffe.proto will be frozen as caffe0.proto. A translation script from caffe0.proto to the new caffe.proto should be implemented to help migrate models.

Don't fret, this is only a one-time break to fix the schema for the 1.0 release.

@sguada
Copy link
Contributor

sguada commented Mar 17, 2014

@jeffdonahue @shelhamer I'm working in refactoring DataLayer into DataSource and DataPreprocessing, and changing caffe.proto. If you work on this let me know so we can coordinate.

@jeffdonahue
Copy link
Contributor

I'm working on this, will probably have some commits pushed tonight or tomorrow.

@shelhamer
Copy link
Member Author

Closing, since this is accomplished by #219.

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

No branches or pull requests

3 participants