Skip to content

Commit

Permalink
Update of convertTrack in DelphesEDM4HepConverter.cc: take the track'…
Browse files Browse the repository at this point in the history
…s omega directly from the curvature of the delphes track, instead of recomputing it from its pT.
  • Loading branch information
EmanuelPerez committed Oct 9, 2023
1 parent b3e12de commit e60aa2d
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions converter/src/DelphesEDM4HepConverter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -556,6 +556,11 @@ namespace k4SimDelphes {
trackState.phi = cand->Phi;
// Same thing under different name in Delphes
trackState.tanLambda = cand->CtgTheta;

/*
// Instead of recomputing the track's omega from its pT, better take it directly
// from the curvature of the delphes track - see below.
// Only do omega when there is actually a magnetic field.
double varOmega = 0;
if (magFieldBz) {
Expand All @@ -567,6 +572,7 @@ namespace k4SimDelphes {
// constant B-field -> relative error on pT is relative error on omega
varOmega = cand->ErrorPT * cand->ErrorPT / cand->PT / cand->PT * trackState.omega * trackState.omega;
}
*/

// fill the covariance matrix. There is a conversion of units
// because the covariance matrix in delphes is with the original units
Expand All @@ -579,6 +585,8 @@ namespace k4SimDelphes {
// relating to curvature (index 2)
double scale2 = -2.; // CAREFUL: DELPHES USES THE HALF-CURVATURE

trackState.omega = cand->C * scale2 ;

covMatrix[0] = covaFB(0, 0);

covMatrix[1] = covaFB(1, 0);
Expand Down

0 comments on commit e60aa2d

Please sign in to comment.