search

snuspl / dolphin

14 stars 2 forks source link

Convolutional layer full implementation #174

Closed ghost closed 8 years ago

ghost commented 8 years ago

This PR includes full implementation of convolutional layers. im2col() and col2im() are implemented to facilitate matrix multiplication in feedForward(), backPropagate() and generateParameterGradient(). Convolutional layer unit test class is added, too. This layer supports only 2D inputs and support for 3D inputs will be implemented in later PR.

beomyeol commented 8 years ago

@seyounglee95 I'll do a pass.

ghost commented 8 years ago

@beomyeol I made new branch issue-171-convolutional_layer_full_implementation_temp. Please check this.

ghost commented 8 years ago

@beomyeol Thank you for your review. I pushed a commit. Please take a look.

beomyeol commented 8 years ago

@seyounglee95 This looks good. However, there are a couple of minor issues. I'll fix them and merging this.