Draft: Test Di-Photon L1Topo Chains for Reprocessing
This MR tracks the implementation of the following di-EM L1 Topo chains:
0INVM70-0DPHI12-eEM9sl1-eEM9sl6 --> DPHI12-M70-2eEM9L
0INVM70-0DPHI12-eEM12sm1-eEM12sm6 --> DPHI12-M70-2eEM12L
0INVM70-2DR15-eEM9sl1-eEM9sl6 --> 2DR15-M70-2eEM9L
0INVM70-2DR15-eEM12sl1-eEM12sl6 --> 2DR15-M70-2eEM12L
Due to unforeseen difficulties in adding more eEM thresholds we are postponing the implementation of the same algorithms on 2eEM9/12M.
Which are threshold variations on the existing DPHI-M70-2eEMXX items created from the InvariantMassDeltaPhiInclusive2 algorithm.
Tagging @khoo
Merge request reports
Activity
added 23.0 TriggerMenu labels
added 1 commit
- 5c65bc0e - Fixed Multiplicity issues for eEM9M and eEM12M
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
added Trigger changes-trigger-counts review-pending-level-1 labels
-
-
Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
-
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
-
-
Athena externals cmake make tests 
Full details available on this CI monitor view. Check the JIRA CI status board for known problems
added 1 commit
- bd6e0339 - Restoring spare thresholds in TopoAlgoDefMultiplicity
-
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
-
-
Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Hi @mmontell. Several CI tests are failing with the same error: "ERROR Algorithm of name Mult_eEM9M in category multiplicity topo is not registered."
Can you look into it?
-- James from L1
added review-user-action-required label and removed review-pending-level-1 label
-
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
added review-pending-level-1 label and removed review-user-action-required label
-
-
Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
removed review-pending-level-1 label
added 1 commit
- bc21a684 - Restored eEM SPARE thresholds in TopoAlgoDefMultiplicity
added 1 commit
- 9702ba0b - Removing DiPhoton topos on eEM9M and eEM12MÂ, moving to L inputs
163 163 # Photon chains for TLA 164 164 ChainProp(name='HLT_g35_loose_PhysicsTLA_L1EM22VHI',stream=['TLA'], groups=SinglePhotonGroup+DevGroup), 165 165 166 # Photons for diPhoton TLA 167 # M, DPHI 168 ChainProp(name='HLT_2g10_loose_PhysicsTLA_L1DPHI12-M70-2eEM9L',l1SeedThresholds=['eEM9'],stream=['TLA'], groups=SupportPhIGroup+Topo2Group+DevGroup), 169 ChainProp(name='HLT_2g10_medium_PhysicsTLA_L1DPHI12-M70-2eEM9L',l1SeedThresholds=['eEM9'],stream=['TLA'], groups=SupportPhIGroup+Topo2Group+DevGroup), 170 ChainProp(name='HLT_2g13_loose_PhysicsTLA_L1DPHI12-M70-2eEM12L',l1SeedThresholds=['eEM12L'],stream=['TLA'], groups=SupportPhIGroup+Topo2Group+DevGroup), -
@khoo How bad is it to have eEM9 as l1SeedThreshold for the eEM9L and eEM12L (not the L) for the 2eEM12 chains?
This is because eEM9L apparently doesn't exist as HLT-l1SeedThreshold, nor does eEM12 plain.
Edited by Marco Montella -
Ok this is sort of interesting/subtle. L1Topo builds the input lists via a defined set of algs. These needn't correspond to the thresholds that are counted by the multiplicity algs.
In particular the loose
eEMinput list is defined here: https://gitlab.cern.ch/atlas/athena/-/blob/23.0/Trigger/TriggerCommon/TriggerMenuMT/python/L1/Config/TopoAlgoDef.py#L80 It doesn't have to reference a concrete threshold. Same for theeEM12. The et threshold is determined in the topo algorithm.Tbh this isn't something I was consciously aware of until you brought it up... But it is apparent (though arguably obscurely so) from TopoAlgoDef.
For HLT seeding, the 'only' drawback is some extra HLT CPU needed to start the reconstruction on the looser thresholds. There should be no technical problem.
added 2063 commits
-
8444d6e3...5d0827d8 - 2048 commits from branch
atlas:23.0 - 8d19bac4 - Implementation of additional eFEX L1 Topo under the INVM_DPHI_eEMsl6 algorithm + chains in Dev Menu
- 3f94ac03 - Implementation of additional eFEX L1 Topo under the INVM_DPHI_eEMsl6 algorithm + chains in Dev Menu
- ee09c94d - Moving new items to MC menu
- 1ae91e14 - Added boosted diPhoton L1s to MC menu
- 9b2e83dc - Adding Additional eEM-M thresholds + DR+M di-eEM L1Topo Algo
- 7f90b9f5 - Trying to correctly define the new L1 algos in the MC Menus
- 349a65d4 - Fixed Multiplicity issues for eEM9M and eEM12M
- 59e42ab7 - Updated references
- aafef408 - eEMSPAREs now onl replaced in MC menu
- c479e6ec - Restoring spare thresholds in TopoAlgoDefMultiplicity
- c8b4eb5f - Exploring solution for TopoAlgoDef
- 1f9c4934 - Restored eEM SPARE thresholds in TopoAlgoDefMultiplicity
- 4e77620b - Removing DiPhoton topos on eEM9M and eEM12MÂ, moving to L inputs
- ee565832 - Cleanup
- 4fe0f7b3 - Trying to merge things
Toggle commit list-
8444d6e3...5d0827d8 - 2048 commits from branch
-
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
-
-
Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
added 349 commits
-
4fe0f7b3...b8c9690f - 320 commits from branch
atlas:23.0 - b8c9690f...e8fdea7a - 19 earlier commits
- 33e81e14 - Trying to correctly define the new L1 algos in the MC Menus
- 8023ee97 - Fixed Multiplicity issues for eEM9M and eEM12M
- 8279dfb6 - Updated references
- 0b591503 - eEMSPAREs now onl replaced in MC menu
- 2e5dbf56 - Restoring spare thresholds in TopoAlgoDefMultiplicity
- c5c24603 - Exploring solution for TopoAlgoDef
- e5ac78e0 - Restored eEM SPARE thresholds in TopoAlgoDefMultiplicity
- d4de8ed5 - Removing DiPhoton topos on eEM9M and eEM12MÂ, moving to L inputs
- 23349986 - Cleanup
- aa137306 - Fixing conflicts pre-merge
Toggle commit list-
4fe0f7b3...b8c9690f - 320 commits from branch
-
This merge request affects 3 packages:
- Trigger/TrigValidation/TrigAnalysisTest
- Trigger/TrigValidation/TrigP1Test
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@martindl ,@ademaria ,@salderwe ,@ggonella ,@vmartin ,@okumura ,@carquin ,@iriu ,@malconad ,@shanisch ,@gipezzul ,@lidiaz as watchers
-
-
Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
