Skip to content

Update residual-layers.lua to be consistent with other torch nn models - #6

Open
lim0606 wants to merge 3 commits into
gcr:masterfrom
lim0606:master
Open

Update residual-layers.lua to be consistent with other torch nn models#6
lim0606 wants to merge 3 commits into
gcr:masterfrom
lim0606:master

Conversation

@lim0606

@lim0606 lim0606 commented Feb 2, 2016

Copy link
Copy Markdown

Hi,

I updated residual-layers.lua so that it can be used with other available layers in torch community.

I thought this might be useful for someone who might want to use the residual layer in their network architecture.

This might be not the preferred way of @gcr.

Anyway, thank you for sharing your review work.

Best regards,

Jaehyun


This change is Reviewable

@gcr

gcr commented Feb 2, 2016

Copy link
Copy Markdown
Owner

I see! Thank you -- I wasn't sure how to express this network in the usual Torch way, so I had to use the nngraph module. This example makes that clearer.

@lim0606

lim0606 commented Feb 2, 2016

Copy link
Copy Markdown
Author

I just meant that the modification intended to make the layer be look-alike some tutorials in https://github.com/torch/torch7/wiki/Cheatsheet, e.g. https://github.com/soumith/imagenet-multiGPU.torch.

Thank you :)

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.

2 participants