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

3D Convolution Layer #2283

Closed
wants to merge 17 commits into from
Closed

3D Convolution Layer #2283

wants to merge 17 commits into from

Conversation

ddetone
Copy link

@ddetone ddetone commented Apr 8, 2015

Here is a layer that implements 3D convolution. I essentially used a lot of the 2D convolution code as a guideline, and extended it to 3 dimensions. My implementation uses a vol2col utility function that is very similar the im2col (wasteful on memory but simple and pretty fast).

The layer, "Convolution3DLayer" inherits directly from "Layer", rather than "BaseConvolutionLayer".

Feedback appreciated! I'd be happy to move stuff around to have it fit best how everyone sees fit.

Next up:
-3D Max Pooling

@shelhamer
Copy link
Member

@ddetone have a look at the N-D convolution in #2049 and compare -- is there an advantage to your layer in the 3-D case? Thanks for sending a PR.

There isn't any 3-D or N-D pooling yet, so that will be a welcome improvement.

@ddetone
Copy link
Author

ddetone commented Apr 13, 2015

I didn't know about the N-D convolution branch. Cool! The implementation in #2049 is certainly more elegant than mine. I'll experiment with the N-D convolution branch and see if I can compare performance.

@shelhamer
Copy link
Member

Closed in favor of #2049, further tuned in #3452.

@shelhamer shelhamer closed this Feb 25, 2016
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.

None yet

2 participants