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
update the lbfgs optimizer #2337
Conversation
Changes Unknown when pulling c3f908e on yorkerlin:develop into * on shogun-toolbox:develop*. |
@@ -391,6 +391,31 @@ typedef int (*lbfgs_progress_t)( | |||
int ls | |||
); | |||
|
|||
/** | |||
* Callback interface to adjust step size based on constrains. |
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.
typo "constraints"
Very nice! |
const float64_t strict_scale=1e-5; | ||
|
||
//Init the parameters used for L-BFGS | ||
lbfgs_parameter_t inti_lbfgs_parameters() |
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.
typo init
@karlnapf |
Thanks! :) |
@karlnapf
Please merge this first.
I will send additional two PRs for dual likelihoods and dual KL method respectively once this is merged.