Presently armijo will update even if we have a bad step. Line 149 of armijo should be moved into the block in lines 132-135. Otherwise we will take a step even if we can't find an acceptable step size and hit block 112-115. See backtrack lines 114-116 as an example.
In addition, I would vote for changing the alpha on line 112 to something more reasonable like 1e-3 or 1e-4.
Presently armijo will update even if we have a bad step. Line 149 of armijo should be moved into the block in lines 132-135. Otherwise we will take a step even if we can't find an acceptable step size and hit block 112-115. See backtrack lines 114-116 as an example.
In addition, I would vote for changing the alpha on line 112 to something more reasonable like 1e-3 or 1e-4.
This should be addressed before #425 #31