Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

Operator modules #416

Closed
hgaiser opened this issue Apr 25, 2017 · 5 comments
Closed

Operator modules #416

hgaiser opened this issue Apr 25, 2017 · 5 comments

Comments

@hgaiser
Copy link
Contributor

hgaiser commented Apr 25, 2017

A big caveat in Caffe's modularity is that it didn't allow layers to be added dynamically, through modules. I had spent some time making a system for this in Caffe and it is currently under review. I have been using it for some time now and it makes adding a new layer very easy. (vanilla) Caffe is installed as a system dependency (with the layer modules PR) and if I need a 'third-party' layer from a PR or some project, I wrap it in its own CMake project, create a module and it's good to go. That way I can run for instance Faster RCNN using upstream Caffe instead of the outdated Faster RCNN version of Caffe.

Looking at the documentation it seems that Caffe2 is suffering from the exact same caveat. I've been going through the documentation but it seems that in order to add a new operator one must clone Caffe2, add the operator and then compile and deploy that special version of Caffe2?

I could try to implement a similar system as with Caffe layer modules, but then Caffe2's operator modules. I was hoping to get some feedback on the suggestion before I spend time on this however.

@slayton58
Copy link
Contributor

Take a look at this repo: caffe2/caffe2_bhtsne - it sounds like an example of what you're after.

@hgaiser
Copy link
Contributor Author

hgaiser commented Apr 26, 2017

Take a look at this repo: caffe2/caffe2_bhtsne - it sounds like an example of what you're after.

Seems you are correct! Thank you for your response.

@hgaiser hgaiser closed this as completed Apr 26, 2017
@nnop
Copy link

nnop commented Mar 15, 2018

Could it also apply to caffe? @slayton58 @hgaiser

@hgaiser
Copy link
Contributor Author

hgaiser commented Mar 15, 2018

@nnop , big sigh...

BVLC/caffe#5294

@nnop
Copy link

nnop commented Mar 15, 2018

Thanks for your contribution and pointing out.

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

No branches or pull requests

3 participants