/caffe

Groomed version of @fyu's dilated convolution #3487

Merged
merged 6 commits into from Dec 28, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@longjon @fyu @tamakoji
@longjon
Contributor

longjon commented Dec 28, 2015

This is just #3452 from @fyu with the following changes which I've made to save a RTT:

  • History has been groomed to squash style fixes, and so that modified files make sense with respect to commit messages.
  • Per #3452 (comment), the literal 10 has been replaced with num_spatial_axes.
  • The caffe.proto comment description of the dilation parameter has been edited to be more definitional and succinct.
  • DeconvolutionLayer::compute_output_shape has been upgraded to support dilated deconv.

I'm sending this as a separate PR for Travis to check, to demonstrate a (relatively) cleaned-up history, and for approval by @fyu.

I'll merge pending Travis and @fyu.

longjon referenced this pull request Dec 28, 2015

Closed

Add Support for Dilated Convolution #3452

@fyu
Contributor

fyu commented Dec 28, 2015

@longjon This looks great to me. Thanks!

@longjon longjon added a commit that referenced this pull request Dec 28, 2015

@longjon longjon Merge pull request #3487 from longjon/dilation 
An early implementation of this functionality for Caffe was written by @gpapan, which was extended and improved by @tamakoji in a previous implementation of this branch.
 08c5dfd

@longjon longjon merged commit 08c5dfd into BVLC:master Dec 28, 2015
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tamakoji

tamakoji commented Dec 29, 2015

👍

fyu referenced this pull request in tensorflow/tensorflow Apr 8, 2016

Closed

feature request: "a trou" (with hole algorithm) #1815

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