Add LRT electrons and muons to PHYSVAL DAOD

Adds LRT electrons and muons to PHYSVAL DAOD. Switch to using decorations for LRT electron ID when running on a derivation, and fall back to computing with the ASG tool otherwise.

This resolves the crash in ATLPHYSVAL-932 and adds LRT muons in anticipation of ATLASMCP-117.

I've tested using 1000 events of /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/DerivationFrameworkART/valid1.601229.PhPy8EG_A14_ttbar_hdamp258p75_SingleLep.recon.AOD.e8472_s3992_s3993_r14133/AOD.31100617._000001.pool.root.1 Producing a PHYSVAL DAOD and then the ntup physval with --validationFlags doEgamma.

The test DAOD event size went from 435.57 kb/evt to 439.21 kb/evt

cc: @gcallea and @wbalunas

added 1 commit

• 8519226b - Cleanup whitespace change

Compare with previous version

marked this merge request as ready

This merge request affects 2 packages:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhysicsValidation
• Reconstruction/egamma/EgammaPhysValMonitoring

Affected files list will not be printed in this case

Adding @zmarshal ,@jcatmore ,@calpigia ,@dshope ,@duperrin ,@goetz as watchers

added 23.0 Derivation Egamma Reconstruction review-pending-level-1 labels

CI Result SUCCESS (hash 8519226b)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 69413]

The changes look okay and there are no CI issues. Approving from L1.

added review-approved label and removed review-pending-level-1 label

added review-approved-point1 label

added review-approved-tier0 label

mentioned in commit a0808287

merged

mentioned in merge request !62244 (merged)