-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Dimensions #75
base: master
Are you sure you want to change the base?
Dimensions #75
Conversation
xscale = net.layers{l}.xscale; | ||
yscale = net.layers{l}.yscale; | ||
|
||
z = convn(net.layers{l - 1}.a{j}, ones(xscale, yscale) / (xscale*yscale), 'valid'); % !! replace with variable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please calculate ones(xscale, yscale) / (xscale*yscale) in cnnsetup, put it in the net somewhere (net._meanFilter or similar), use that and remove the comment to replace with var
Moved the averaging filter definition from cnnff to cnnsetup. Now it is computed only once, during the net setup.
Funky.. #76 implements non-square kernels. Did you guys coordinate? Anyway, same comments apply. Sorry about the extra feedback btw. I was really busy last time I reviewed. |
Any updates on this? |
Hi guys, I believe this is already working in a branch of my clone of the repo. Let On 11 May 2014 10:01, Rasmus Berg Palm [email protected] wrote:
José Pablo Alberto Andreotti. |
What's the status? |
Really simple change to allow the dimensions of the input to be different. In this way, one of the dimensions can be seen as time, so you can build a TDNN(time delay neural network) to work with speech signals.