Fix clippy warnings
This commit is contained in:
@@ -50,14 +50,14 @@ impl<T: RealNumber> FirstOrderOptimizer<T> for GradientDescent<T> {
|
||||
let f_alpha = |alpha: T| -> T {
|
||||
let mut dx = step.clone();
|
||||
dx.mul_scalar_mut(alpha);
|
||||
f(&dx.add_mut(&x)) // f(x) = f(x .+ gvec .* alpha)
|
||||
f(dx.add_mut(&x)) // f(x) = f(x .+ gvec .* alpha)
|
||||
};
|
||||
|
||||
let df_alpha = |alpha: T| -> T {
|
||||
let mut dx = step.clone();
|
||||
let mut dg = gvec.clone();
|
||||
dx.mul_scalar_mut(alpha);
|
||||
df(&mut dg, &dx.add_mut(&x)); //df(x) = df(x .+ gvec .* alpha)
|
||||
df(&mut dg, dx.add_mut(&x)); //df(x) = df(x .+ gvec .* alpha)
|
||||
gvec.dot(&dg)
|
||||
};
|
||||
|
||||
@@ -66,7 +66,7 @@ impl<T: RealNumber> FirstOrderOptimizer<T> for GradientDescent<T> {
|
||||
let ls_r = ls.search(&f_alpha, &df_alpha, alpha, fx, df0);
|
||||
alpha = ls_r.alpha;
|
||||
fx = ls_r.f_x;
|
||||
x.add_mut(&step.mul_scalar_mut(alpha));
|
||||
x.add_mut(step.mul_scalar_mut(alpha));
|
||||
df(&mut gvec, &x);
|
||||
gnorm = gvec.norm2();
|
||||
}
|
||||
|
||||
@@ -117,14 +117,14 @@ impl<T: RealNumber> LBFGS<T> {
|
||||
let f_alpha = |alpha: T| -> T {
|
||||
let mut dx = state.s.clone();
|
||||
dx.mul_scalar_mut(alpha);
|
||||
f(&dx.add_mut(&state.x)) // f(x) = f(x .+ gvec .* alpha)
|
||||
f(dx.add_mut(&state.x)) // f(x) = f(x .+ gvec .* alpha)
|
||||
};
|
||||
|
||||
let df_alpha = |alpha: T| -> T {
|
||||
let mut dx = state.s.clone();
|
||||
let mut dg = state.x_df.clone();
|
||||
dx.mul_scalar_mut(alpha);
|
||||
df(&mut dg, &dx.add_mut(&state.x)); //df(x) = df(x .+ gvec .* alpha)
|
||||
df(&mut dg, dx.add_mut(&state.x)); //df(x) = df(x .+ gvec .* alpha)
|
||||
state.x_df.dot(&dg)
|
||||
};
|
||||
|
||||
@@ -206,7 +206,7 @@ impl<T: RealNumber> FirstOrderOptimizer<T> for LBFGS<T> {
|
||||
) -> OptimizerResult<T, X> {
|
||||
let mut state = self.init_state(x0);
|
||||
|
||||
df(&mut state.x_df, &x0);
|
||||
df(&mut state.x_df, x0);
|
||||
|
||||
let g_converged = state.x_df.norm(T::infinity()) < self.g_atol;
|
||||
let mut converged = g_converged;
|
||||
|
||||
Reference in New Issue
Block a user