tree 43ed3193d52756172de4aadf27de8baeeb2e547b
parent 9631c693542b95c4fa5a93dfb99dcf41e03bcab8
author Sameer Agarwal <sameeragarwal@google.com> 1417324783 -0800
committer Sameer Agarwal <sameeragarwal@google.com> 1417384310 -0800

Do not try the gradient step if TR step line search fails.

The line search used by the trust region minimizer when enforcing
the bounds constraints starts by using the trust region step
as the line search direction and if that fails, uses the gradient
as the fallback.

The problem with this logic is that the calling code only sees
whether one of the line searches succeeds or not. It does not see
that the fallback happened. So if the fallback line search suceeeds
it still thinks that the line search direction was the trust region
step. This is clearly wrong.

This change, removes the broken fallback logic. This has no effect
on current solution quality as it stands.

Change-Id: Ibc8edd98f77c782ec4708d1e66eaa76d6867b990