Skip to content
Snippets Groups Projects
Commit 8b42d708 authored by Adam Edward Barton's avatar Adam Edward Barton
Browse files

Merge branch 'drop_EgammaCovarianceCalc_since_suppressed_in_AOD' into 'main'

Drop EgammaCovarianceCalculation since suppressed in_AOD

See merge request !71636
parents 192ea0b8 db943456
No related branches found
No related tags found
29 merge requests!78241Draft: FPGATrackSim: GenScan code refactor,!78236Draft: Switching Streams https://its.cern.ch/jira/browse/ATR-27417,!78056AFP monitoring: new synchronization and cleaning,!78041AFP monitoring: new synchronization and cleaning,!77990Updating TRT chip masks for L1TRT trigger simulation - ATR-28372,!77733Draft: add new HLT NN JVT, augmented with additional tracking information,!77731Draft: Updates to ZDC reconstruction,!77728Draft: updates to ZDC reconstruction,!77522Draft: sTGC Pad Trigger Emulator,!76725ZdcNtuple: Fix cppcheck warning.,!76611L1CaloFEXByteStream: Fix out-of-bounds array accesses.,!76475Punchthrough AF3 implementation in FastG4,!76474Punchthrough AF3 implementation in FastG4,!76343Draft: MooTrackBuilder: Recalibrate NSW hits in refine method,!75729New implementation of ZDC nonlinear FADC correction.,!75703Draft: Update to HI han config for HLT jets,!75184Draft: Update file heavyions_run.config,!74430Draft: Fixing upper bound for Delayed Jet Triggers,!73963Changing the path of the histograms to "Expert" area,!73875updating ID ART reference plots,!73874AtlasCLHEP_RandomGenerators: Fix cppcheck warnings.,!73449Add muon detectors to DarkJetPEBTLA partial event building,!73343Draft: [TrigEgamma] Add photon ringer chains on bootstrap mechanism,!72336Fixed TRT calibration crash,!72176Draft: Improving L1TopoOnline chain that now gets no-empty plots. Activating it by default,!72012Draft: Separate JiveXMLConfig.py into Config files,!71876Fix MET trigger name in MissingETMonitoring,!71820Draft: Adding new TLA End-Of-Fill (EOF) chains and removing obsolete DIPZ chains,!71636Drop EgammaCovarianceCalculation since suppressed in_AOD
...@@ -122,7 +122,6 @@ def createEgammaConfigFlags(): ...@@ -122,7 +122,6 @@ def createEgammaConfigFlags():
lambda prevFlags: ( lambda prevFlags: (
prevFlags.Egamma.Keys.Output.ElectronsSuppESD + '.' + prevFlags.Egamma.Keys.Output.ElectronsSuppESD + '.' +
prevFlags.Egamma.Keys.Output.EgammaSuppAOD + '.' prevFlags.Egamma.Keys.Output.EgammaSuppAOD + '.'
"-EgammaCovarianceMatrix."
"-isEMLHLoose.-isEMLHTight.-isEMLHMedium.-isEMMedium")) "-isEMLHLoose.-isEMLHTight.-isEMLHMedium.-isEMMedium"))
egcf.addFlag("Egamma.Keys.Output.ForwardElectrons", 'ForwardElectrons') egcf.addFlag("Egamma.Keys.Output.ForwardElectrons", 'ForwardElectrons')
......
...@@ -10,15 +10,6 @@ namespace { ...@@ -10,15 +10,6 @@ namespace {
constexpr float el_mass = 0.510998; constexpr float el_mass = 0.510998;
constexpr float ph_mass = 0.0; constexpr float ph_mass = 0.0;
AmgMatrix(5, 5)
jacobian(const double phi, const int charge, const double momentum) {
AmgMatrix(5, 5) jac;
jac.setIdentity();
jac(3, 3) = -1.* (1. / std::sin(phi)); // deta/dtheta
jac(4, 4) = -1 * charge * (momentum * momentum); // dp/d(q/p)
return jac;
}
void void
setFromCluster(xAOD::Egamma& eg) setFromCluster(xAOD::Egamma& eg)
{ {
...@@ -51,34 +42,6 @@ setFromTrkCluster(xAOD::Electron& el) ...@@ -51,34 +42,6 @@ setFromTrkCluster(xAOD::Electron& el)
const double pt = const double pt =
E > el_mass ? sqrt(E * E - el_mass * el_mass) / cosh(eta) : 0; E > el_mass ? sqrt(E * E - el_mass * el_mass) / cosh(eta) : 0;
el.setP4(pt, eta, phi, el_mass); el.setP4(pt, eta, phi, el_mass);
// Electron with tracks all should have a covariance matrix set
AmgMatrix(4, 4) matrix;
matrix.setZero();
static const eg_resolution eg_resol("run2_pre");
const float sigmaE_over_E = eg_resol.getResolution(el);
matrix(0, 0) =
(sigmaE_over_E * E * sigmaE_over_E * E) / (cosh(eta) * cosh(eta));
// Direction can be filled only when a good track is there
if (goodTrack) {
xAOD::ParametersCovMatrix_t covmat =
trackParticle->definingParametersCovMatrix();
// Start from the d0, z0, phi, theta, q/p
// and make the (Et, eta, phi, M)
AmgMatrix(5, 5) J = jacobian(trackParticle->phi(), trackParticle->charge(),
trackParticle->p4().P());
AmgMatrix(5, 5) m;
m.setZero();
m = J * (covmat * J.transpose());
matrix(1, 1) = m(3, 3);
matrix(2, 2) = m(2, 2);
matrix.fillSymmetric(0, 1, m(4, 3));
matrix.fillSymmetric(0, 2, m(4, 2));
matrix.fillSymmetric(1, 2, m(3, 2));
}
el.setCovMatrix(matrix.cast<float>());
} }
void void
...@@ -110,7 +73,7 @@ calculate(xAOD::Electron& electron) { ...@@ -110,7 +73,7 @@ calculate(xAOD::Electron& electron) {
} }
} }
void void
calculate(xAOD::Photon& photon) { calculate(xAOD::Photon& photon) {
if (xAOD::EgammaHelpers::conversionType(&photon) == if (xAOD::EgammaHelpers::conversionType(&photon) ==
xAOD::EgammaParameters::doubleSi) { xAOD::EgammaParameters::doubleSi) {
......
...@@ -24,10 +24,10 @@ references_map = { ...@@ -24,10 +24,10 @@ references_map = {
"d1759": "v15", "d1759": "v15",
"d1912": "v5", "d1912": "v5",
# Reco # Reco
"q442": "v51", "q442": "v52",
"q449": "v81", "q449": "v82",
"q452": "v11", "q452": "v12",
"q454": "v17", "q454": "v18",
# Derivations # Derivations
"data_PHYS_Run2": "v23", "data_PHYS_Run2": "v23",
"data_PHYSLITE_Run2": "v4", "data_PHYSLITE_Run2": "v4",
......
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