)]}'
{
  "commit": "9438c370f58ae383e868bf176c65a359bc9cc45e",
  "tree": "eceea4f5b9a3756090c3e3014c9093c8b9ad03ca",
  "parents": [
    "c964fce90ce90d6fa61553d42ab1042a177ab200"
  ],
  "author": {
    "name": "Sameer Agarwal",
    "email": "sameeragarwal@google.com",
    "time": "Thu Sep 22 17:28:52 2022 -0700"
  },
  "committer": {
    "name": "Sameer Agarwal",
    "email": "sameeragarwal@google.com",
    "time": "Thu Sep 22 17:28:52 2022 -0700"
  },
  "message": "Restore the semantics of TrustRegionMinimizer\n\nA previous change to fix a bug related to how x_norm was being\ninitialized changed the semantics of the TrustRegionMinimizer\nloop. With the bug, the parameter_tolerance was being ignored\ntill the first successful step was encountered.\n\nparameter_tolerance based convergence is a hack anyways, so\nrestoring the previous semantics is reasonable and will preserve\nexisting code/tests that depend on this behaviour.\n\nChange-Id: Ia00ca6c47f77e74dee64ad746f4299a9eab0eb7e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "60f6b6291f8df28a675f2309ec26f2c900af0a57",
      "old_mode": 33188,
      "old_path": "internal/ceres/trust_region_minimizer.cc",
      "new_id": "800b6769ed5c4abc7c23a20cc4e96f778862d996",
      "new_mode": 33188,
      "new_path": "internal/ceres/trust_region_minimizer.cc"
    }
  ]
}