Closed josyoun closed 8 years ago
This appears to be fixed, but I'm at a loss at when I did it. Now, alpha0
is the maximum that we'll search. Basically, this came up in the new interior point code checked it at commit 9426f18e1ed26f1996de5a3cde21dddd2984389e since we have to coordinate the safeguard with alpha0
. Anyway, tracking down exactly when this happened isn't that important to me and it's fixed, so we'll close out this issue.
Right now,on line search methods, we search a distance of 2 alpha0 in front of the current step. It's not clear to me why this isnt' just alpha0. Basically, it's causing me grief when trying to set an initial alpha0 since I normally think of how big I want the intial steps to be and I always forget the factor of 2. If it's not important, we should really just make the maximum search be alpha0.