From 4c02d874932c3162e03633ae7d6535b9a92b6be7 Mon Sep 17 00:00:00 2001
From: Federico Montesino Pouzols <federico.montesino-pouzols@stfc.ac.uk>
Date: Tue, 13 Oct 2015 17:52:45 +0100
Subject: [PATCH] check retun value from doFitFunction, 1325901, re #13951

---
 .../Algorithms/RefinePowderInstrumentParameters3.cpp  | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/Framework/CurveFitting/src/Algorithms/RefinePowderInstrumentParameters3.cpp b/Framework/CurveFitting/src/Algorithms/RefinePowderInstrumentParameters3.cpp
index a5c35c1f331..f87a83a0013 100644
--- a/Framework/CurveFitting/src/Algorithms/RefinePowderInstrumentParameters3.cpp
+++ b/Framework/CurveFitting/src/Algorithms/RefinePowderInstrumentParameters3.cpp
@@ -870,8 +870,15 @@ double RefinePowderInstrumentParameters3::calculateFunctionError(
   // 2. Fit with zero iteration
   double chi2;
   string fitstatus;
-  doFitFunction(function, dataws, wsindex, "Levenberg-MarquardtMD", 0, chi2,
-                fitstatus);
+  const std::string minimizer = "Levenberg-MarquardtMD";
+  bool fitOK =
+      doFitFunction(function, dataws, wsindex, minimizer, 0, chi2, fitstatus);
+
+  if (!fitOK) {
+    g_log.warning() << "Fit by " << minimizer
+                    << " with 0 iterations failed, with reason: " << fitstatus
+                    << "\n";
+  }
 
   // 3. Restore the fit/fix setup
   for (size_t i = 0; i < parnames.size(); ++i) {
-- 
GitLab