Optimization of Run-2 TGC-BW Coincidence Window

The storing method of the contents of Coincidence Windows for TGC Big-Wheel is changed for more optimization of memory usage by TGCTriggerDbAlg.
This is one of goal for ATR-21107.

As suggested by the last comment in !38485 (merged), a single std::unordered_map is adopted instead of a load of std::map instances. The unordered_map was faster and smaller memory usage when I compared.

The following test is successfully finished:

RecExCommon_links.sh; athena.py myTopOptions.py

Reco_tf.py --AMI q221 --inputRDOFile /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/TriggerTest/valid1.410000.PowhegPythiaEvtGen_P2012_ttbar_hdamp172p5_nonallhad.merge.RDO.e4993_s3214_r11315/RDO.17533168._000001.pool.root.1 --maxEvents 100 --athenaopts='--threads=1 --pmon=sdmonfp' --outputRDO_TRIGFile=myRDOTrigger.pool.root --steering=doRDO_TRIG --preExec all:"setMenu='LS2_v1_TriggerValidation_mc_prescale'"

Moreover, I confirmed to output same result of pT determination by the single muon sample.

added 1 commit

• b1ae9faa - fix the indexing of pT value

Compare with previous version

unmarked as a Work In Progress

changed the description

This merge request affects 2 packages:

• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondSvc
• Trigger/TrigT1/TrigT1TGC

Affected files list will not be printed in this case

Adding @rosati ,@wleight ,@kwhalen ,@nkoehler ,@afaulkne as watchers

added MuonSpectrometer Trigger master review-pending-level-1 labels

CI Result SUCCESS (hash b1ae9faa)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 25196]

added review-user-action-required label

added 1 commit

• 07a6589a - add const variables and comments for hard-coded numbers

Compare with previous version

This merge request affects 2 packages:

• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondSvc
• Trigger/TrigT1/TrigT1TGC

Affected files list will not be printed in this case

Adding @rosati ,@wleight ,@kwhalen ,@nkoehler ,@afaulkne as watchers

removed review-user-action-required label

CI Result SUCCESS (hash 07a6589a)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 25282]

resolved all threads

Seems fine to me now, approving.

Pavol [as L2 MR shifter]

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

merged

mentioned in commit 8bb8f885