Skip to content
Snippets Groups Projects
Commit 0abc0fbe authored by Xiaocong Ai's avatar Xiaocong Ai Committed by Paul Gessinger
Browse files

Revise the KF error message

parent 96d5cafe
No related branches found
No related tags found
1 merge request!793Fix kf surface check
Pipeline #1498710 passed
......@@ -546,6 +546,8 @@ class KalmanFitter {
// We count the processed state
++result.processedStates;
} else if (surface->surfaceMaterial() != nullptr) {
// We only create track states here if there is already measurement
// detected
if (result.measurementStates > 0) {
// No source links on surface, add either hole or passive material
// TrackState entry multi trajectory. No storage allocation for
......@@ -991,7 +993,6 @@ class KalmanFitter {
/// It could happen that the fit ends in zero processed states.
/// The result gets meaningless so such case is regarded as fit failure.
//@TODO: should we require the number of measurments >0 ?
if (kalmanResult.result.ok() and not kalmanResult.processedStates) {
kalmanResult.result = Result<void>(KalmanFitterError::PropagationInVain);
}
......
......@@ -40,7 +40,7 @@ class KalmanFitterErrorCategory : public std::error_category {
case KalmanFitterError::OutputConversionFailed:
return "Kalman output conversion failed";
case KalmanFitterError::PropagationInVain:
return "No detector observed during the propagation";
return "No measurement detected during the propagation";
default:
return "unknown";
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment