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

Add support to groups in SpatialConvolutionMM #438

Open
fmassa opened this issue Oct 21, 2015 · 5 comments
Open

Add support to groups in SpatialConvolutionMM #438

fmassa opened this issue Oct 21, 2015 · 5 comments

Comments

@fmassa
Copy link
Contributor

fmassa commented Oct 21, 2015

Is there any interest in adding groups support to SpationConvolutionMM ?

Groups behaviour can be simulated by combining a few modules (like Concat + Narrow + SpatialConvolution, for example), but is it worth it adding dedicated C/Cuda paths for this option ?

The only reason why I would see some interest is to allow some Caffe models to work in vanilla nn (like AlexNet).

If there is interest in it, I can do it.

@htoyryla
Copy link

Francisco Massa notifications@github.com kirjoitti 22.10.2015 kello 1.05:

Is there any interest in adding groups support to SpationConvolutionMM ?

Groups behaviour can be simulated by combining a few modules (like Concat + Narrow + SpatialConvolution, for example), but is it worth it adding dedicated C/Cuda paths for this option ?

The only reason why I would see some interest is to allow some Caffe models to work in vanilla nn (like AlexNet).

I am interested in using AlexNet in vanilla nn (without GPU/cuda).

But I am just a newcomer in deep learning so you are free to ignore my wishes.

Hannu Töyrylä
http://liipetti.net/erratic/

If there is interest in it, I can do it.


Reply to this email directly or view it on GitHub.

deltheil referenced this issue in szagoruyko/loadcaffe Jan 4, 2016
@varunagrawal
Copy link

@fmassa the group feature in Caffe and the need to convert caffe models to torch models is what brought me here. If you can add in this feature, it would be amazing!

@fmassa
Copy link
Contributor Author

fmassa commented Apr 20, 2017

@varunagrawal I think the easiest for now would be to add the groups support in the lua level, as done in PyTorch. I unfortunately won't have time in the near future to do it, as I've moved my stack to PyTorch.

@varunagrawal
Copy link

Alright. I wanted to convert some caffe models to torch and then to PyTorch, but the lack of the groups feature in torch is preventing me from doing that. Any ideas on what can I do alternatively?

@varunagrawal
Copy link

Nevermind, found a solution. loadcaffe can take cudnn as an argument for the backend and the torch cudnn bindings support groups. Cheers!

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

No branches or pull requests

3 participants