Skip to content

Commit b69a505

Browse files
committed
CheckResid: initialise sigmaYZ of the PV pseudo-cluster
o2::BaseCluster default constructor leaves mSigmaYZ uninitialised, so with addPVAsCluster the PV cluster entered the refit with a garbage YZ covariance term, making the residuals non-reproducible between identical runs.
1 parent 8cc0b98 commit b69a505

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎Detectors/GlobalTrackingWorkflow/study/src/CheckResidSpec.cxx‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -453,6 +453,7 @@ bool CheckResidSpec::processITSTrack(const o2::its::TrackITS& iTrack, const o2::
453453
bcPV.setSigmaY2(0.5 * (pv.getSigmaX2() + pv.getSigmaY2()));
454454
bcPV.setSigmaZ2(pv.getSigmaZ2());
455455
bcPV.setSensorID(-1);
456+
bcPV.setSigmaYZ(0.f); // not initialised by the BaseCluster default constructor
456457
clArr[0] = &bcPV;
457458
}
458459
// collect all track clusters to array, placing them to layer+1 slot

0 commit comments

Comments
 (0)