JuliaSmoothOptimizers / LinearOperators.jl

Linear Operators for Julia
Other
150 stars 32 forks source link

Verify the LBFGSOperator update (push!) #223

Open paraynaud opened 2 years ago

paraynaud commented 2 years ago

Hello!

Context:

I wonder if there is a way to see if push!(opLBFGS, s, y) performs the update or not. I could check the curvature condition for each (s,y) on my side, but that would be a double check of the cruvature condition.

What are you thoughts?

dpo commented 2 years ago

We could reinstate this: https://github.com/JuliaSmoothOptimizers/LinearOperators.jl/blob/main/src/lbfgs.jl#L272

You can also use damping.