| commit | 4be580dd4b648abdd16915953e0a421a6f7e2e9d | [log] [tgz] |
|---|---|---|
| author | Sameer Agarwal <sameeragarwal@google.com> | Tue Feb 25 21:11:35 2014 -0800 |
| committer | Sameer Agarwal <sameeragarwal@google.com> | Wed Feb 26 11:44:16 2014 -0800 |
| tree | 984a129c95390ca0c82f24258540716330e51c61 | |
| parent | 879fc2827a5b3999544b2446795c834642a79139 [diff] |
Reorder code in LineSearchMinimizer. Change the order in which IterationSummary objects are added to Solver::Summary and when the convergence tests are done. Previously there was a bug which prevented the iteration summary from carrying the correct cost of the current state of the optimization problem as the convergence test triggers before the cost can be updated. Thanks to Thad Hughes for reporting this. Change-Id: Iba0dbb4cb30c9ec79fbc72ec5cf4602b2a0c207b