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.

Sign up for GitHub

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

Jump to bottom

minor changes #2

Open

9il wants to merge 1 commit into ShigekiKarita:master from 9il:patch-2

Contributor

9il commented Oct 23, 2017

btw, should prior use sqrt instead of / 2?


          few changes

41cddc3

btw, should `prior` use `sqrt` instead of `/ 2`

Owner

ShigekiKarita commented Oct 23, 2017

Thanks for PRs.

It's not sqrt but squared L2-norm (see. https://en.wikipedia.org/wiki/Structured_support_vector_machine#Training)
Although 1/2 can be ignored by the hyper-parameter C, I use 1/2 * ||w||^2 because the prior's gradient w.r.t. w is just w.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment