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

Cutsomizable "channel" dimension for ND-tensor #25

Merged
merged 16 commits into from Dec 19, 2014
Merged

Cutsomizable "channel" dimension for ND-tensor #25

merged 16 commits into from Dec 19, 2014

Conversation

pluskid
Copy link
Owner

@pluskid pluskid commented Dec 19, 2014

This is one of the step of generalizing 4D tensor to ND tensor for Mocha. Originally, the 3rd dimension of a 4D tensor is named the channel dimension, and many layers operate explicitly along that dimension (e.g. SoftmaxLayer, SoftmaxLossLayer, AccuracyLayer, etc.). Also, ChannelPoolingLayer is generalized to be a 1D pooling along any user specified dimension.

This PR is an effort for a well-defined behavior under the case of ND tensor. The goal is to allow the user to specify which dimension to operate on for those layers, and use a default dimension that is backward compatible with the good old 4D-tensor world.

@pluskid pluskid mentioned this pull request Dec 19, 2014
4 tasks
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) when pulling 1699b23 on nd-tensor into 44f5311 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) when pulling 0a2c21a on nd-tensor into 44f5311 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) when pulling 5fa3a4e on nd-tensor into 44f5311 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) when pulling c4dc88c on nd-tensor into 44f5311 on master.

@pluskid pluskid changed the title [WIP] Cutsomizable "channel" dimension for ND-tensor Cutsomizable "channel" dimension for ND-tensor Dec 19, 2014
pluskid added a commit that referenced this pull request Dec 19, 2014
Cutsomizable "channel" dimension for ND-tensor
@pluskid pluskid merged commit 44fca43 into master Dec 19, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+0.11%) when pulling 822dabc on nd-tensor into 44f5311 on master.

@pluskid pluskid deleted the nd-tensor branch December 20, 2014 01:55
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