)]}'
{
  "commit": "eafeca5dcb7af8688d40a9c14b0d2fcb856c96fc",
  "tree": "5efbf3f94dd6e77ff6a2b03161670ae1b7d1ec92",
  "parents": [
    "1fd0be916dd4ff4241bd52264b9e9170bc7e4339"
  ],
  "author": {
    "name": "Sameer Agarwal",
    "email": "sameeragarwal@google.com",
    "time": "Mon Sep 28 11:12:59 2020 -0700"
  },
  "committer": {
    "name": "Sameer Agarwal",
    "email": "sameeragarwal@google.com",
    "time": "Mon Sep 28 12:19:45 2020 -0700"
  },
  "message": "Fix a logging bug in TrustRegionMinimizer.\n\nUpon encountering an unsuccessful step (one where the cost goes up)\nthe the trust region minimizer failed to populate the gradient norm\nin the IterationSummary. This would cause the gradient norm to be\nlogged as zero which is incorrect. Instead it should be the gradient\nnorm at the current point.\n\nThis CL fixes this issue.\n\nBefore:\niter      cost      cost_change  |gradient|   |step|    tr_ratio  tr_radius  ls_iter  iter_time  total_time\n   0  1.115206e+07    0.00e+00    1.90e+07   0.00e+00   0.00e+00  1.00e+04        0    2.72e-01    1.33e+00\n   1  3.687552e+06    7.46e+06    1.84e+08   2.86e+03   6.91e-01  1.06e+04        1    1.32e+00    2.65e+00\n   2  3.670266e+10   -3.67e+10    0.00e+00   3.27e+03  -1.07e+04  5.30e+03        1    7.52e-01    3.40e+00\n   3  4.335397e+07   -3.97e+07    0.00e+00   2.74e+03  -1.16e+01  1.32e+03        1    7.28e-01    4.13e+00\n   4  1.345488e+06    2.34e+06    4.12e+07   1.55e+03   6.87e-01  1.40e+03        1    9.31e-01    5.06e+00\n   5  5.376653e+05    8.08e+05    9.99e+06   6.64e+02   7.46e-01  1.59e+03        1    9.64e-01    6.03e+00\n\nAfter:\niter      cost      cost_change  |gradient|   |step|    tr_ratio  tr_radius  ls_iter  iter_time  total_time\n   0  1.115206e+07    0.00e+00    1.90e+07   0.00e+00   0.00e+00  1.00e+04        0    2.37e-01    1.13e+00\n   1  3.687552e+06    7.46e+06    1.84e+08   2.86e+03   6.91e-01  1.06e+04        1    1.08e+00    2.21e+00\n   2  3.670266e+10   -3.67e+10    1.84e+08   3.27e+03  -1.07e+04  5.30e+03        1    7.50e-01    2.96e+00\n   3  4.335397e+07   -3.97e+07    1.84e+08   2.74e+03  -1.16e+01  1.32e+03        1    7.13e-01    3.67e+00\n   4  1.345488e+06    2.34e+06    4.12e+07   1.55e+03   6.87e-01  1.40e+03        1    9.01e-01    4.57e+00\n   5  5.376653e+05    8.08e+05    9.99e+06   6.64e+02   7.46e-01  1.59e+03        1    9.36e-01    5.51e+00\n\nChange-Id: Iae538fe089be07c7bb219337a6f1392f7213acfe\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "7065977ad77d3753e01706d138371c0085f1e2c5",
      "old_mode": 33188,
      "old_path": "internal/ceres/trust_region_minimizer.cc",
      "new_id": "196f9e62342b562aef0e67546751db2b5f2190ab",
      "new_mode": 33188,
      "new_path": "internal/ceres/trust_region_minimizer.cc"
    },
    {
      "type": "modify",
      "old_id": "e18193c92596623549ef1b62c1798752419cfb0c",
      "old_mode": 33188,
      "old_path": "internal/ceres/trust_region_minimizer.h",
      "new_id": "27a1b78487667291460f6eb61f124ee80c4e3ac6",
      "new_mode": 33188,
      "new_path": "internal/ceres/trust_region_minimizer.h"
    }
  ]
}