Be more verbose at errors in AP_amplitude

BlueBrain · Oct 5, 2015 · 55fedb8 · 55fedb8
1 parent 940f04e
commit 55fedb8
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/efel/cppcore/LibV1.cpp b/efel/cppcore/LibV1.cpp
@@ -460,23 +460,25 @@ int LibV1::AP_amplitude(mapStr2intVec& IntFeatureData,
     vector<double> v;
     retVal = getDoubleVec(DoubleFeatureData, StringData, "V", v);
     if (retVal <= 0) {
+      GErrorStr += "AP_amplitude: Can't find voltage vector V";  
       return -1;
     }
     retVal = getDoubleVec(DoubleFeatureData, StringData, "peak_voltage",
                           peakvoltage);
     if (retVal <= 0) {
-      GErrorStr += "Error calculating peak_voltage for AP_amplitude";
+      GErrorStr += "AP_amplitude: Error calculating peak_voltage";
       return -1;
     }
     retVal = getIntVec(IntFeatureData, StringData, "AP_begin_indices",
                        apbeginindices);
     if (retVal <= 0) {
-      GErrorStr += "Error calculating AP_begin_indicies for AP_amplitude";
+      GErrorStr += "AP_amplitude: Error calculating AP_begin_indicies";
       return -1;
     }
 
     if (peakvoltage.size() > apbeginindices.size()) {
-      GErrorStr += "More peakvoltage entries than AP begin entries";
+      GErrorStr += 
+          "AP_amplitude: More peak_voltage entries than AP_begin_indices entries";
       return -1;
     }
 
@@ -488,7 +490,6 @@ int LibV1::AP_amplitude(mapStr2intVec& IntFeatureData,
     setDoubleVec(DoubleFeatureData, StringData, "AP_amplitude", apamplitude);
     return apamplitude.size();
   }
-  return -1;
 }
 
 // AHP_depth_abs