saberbud/ceres-solver
1
0
Fork 0
mirror of https://github.com/ceres-solver/ceres-solver.git synced 2026-08-30 00:50:37 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove overzealous checks in Summary::FullReport.

Browse Source 
Thanks to sebi.koch@gmail.com for reporting this.

Change-Id: I1ba9b375e5cf66639e292ba37b34a90446f13162
This commit is contained in:
Sameer Agarwal
2013-11-25 13:37:02 -08:00
parent 40ef90304a
commit 5794d41be2
1 changed files with 1 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/ceres/solver.cc
+1 -8
View File
@@ -86,6 +86,7 @@ Solver::Summary::Summary()
// accidentally reporting default values.
: minimizer_type(TRUST_REGION),
termination_type(DID_NOT_RUN),
error("ceres::Solve was not called."),
initial_cost(-1.0),
final_cost(-1.0),
fixed_cost(-1.0),
@@ -133,10 +134,6 @@ Solver::Summary::Summary()
string Solver::Summary::BriefReport() const {
string report = "Ceres Solver Report: ";
if (termination_type == DID_NOT_RUN) {
CHECK(!error.empty())
<< "Solver terminated with DID_NOT_RUN but the solver did not "
<< "return a reason. This is a Ceres error. Please report this "
<< "to the Ceres team";
return report + "Termination: DID_NOT_RUN, because " + error;
}
@@ -318,10 +315,6 @@ string Solver::Summary::FullReport() const {
}
if (termination_type == DID_NOT_RUN) {
CHECK(!error.empty())
<< "Solver terminated with DID_NOT_RUN but the solver did not "
<< "return a reason. This is a Ceres error. Please report this "
<< "to the Ceres team";
StringAppendF(&report, "Termination: %20s\n",
"DID_NOT_RUN");
StringAppendF(&report, "Reason: %s\n", error.c_str());