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

Is kNormLimitColumnwise performing the correct reduction? #60

Open
lzamparo opened this issue May 9, 2014 · 0 comments
Open

Is kNormLimitColumnwise performing the correct reduction? #60

lzamparo opened this issue May 9, 2014 · 0 comments

Comments

@lzamparo
Copy link

lzamparo commented May 9, 2014

I looked at your code for max-norm regularization after reading section 2.2 of your thesis (and the papers you cited), and was puzzled by how the CUDA kernel performs the computation.

As I understand it, the max-norm of each column is limited to some value B by ensuring that the column with the maximal sum of squares is re-scaled so that its norm does not exceed B. But your kernel does a sum reduction (c.f line 1068) on the sum of squares of the columns rather than a max reduction. This seems like it will lead to more stringent / severe regularization than was intended by the authors of the max-norm paper.

Am I misunderstanding the code? Or are you using the wrong reduction (sum vs max) here?

Thanks for clearing this up :)

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

No branches or pull requests

1 participant