add all trigger matchable chains for Run3 in DAOD_PHYS (ATR-23142)

this to add electron, muon and tau chains for trigger matching decoration in DAOD_PHYS only when the run3 HLT is used. This is temporary for the small production of validation samples for the run3 trigger menu preparation. This will removed in the Run3 MC bulk production.

For the moment the LRT chains are vetoed because they introduce errors to be understood

@bernius @aranzazu

changed title from WIP: add all trigger matchable chains for Run3 in DAOD_PHYS to WIP: add all trigger matchable chains for Run3 in DAOD_PHYS (ATR-23142)

added 38 commits

• d68a6b4d...b68eee3d - 36 commits from branch atlas:master
• b5e02e47 - Merge remote-tracking branch 'upstream/master' into ATR-triggerListPHYS_dzanzi
• 4fc1af36 - remove duplicated chains

Compare with previous version

@jburr I made two not overlapping list, but not I get an error which I don't how to debug:

AODtoDAOD 17:37:56 2021-03-29 17:37:54,230 PHYSTriggerMatchingToolNoTau_Alg                     FATAL  Standard std::exception is caught
AODtoDAOD 17:37:56 2021-03-29 17:37:54,230 PHYSTriggerMatchingToolNoTau_Alg                     ERROR SG::ExcInvalidLink: Attempt to dereference invalid DataLink / ElementLink [1241842700/] (883500482)
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAlgSeq                                            FATAL  Standard std::exception is caught
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAlgSeq                                            ERROR SG::ExcInvalidLink: Attempt to dereference invalid DataLink / ElementLink [1241842700/] (883500482)
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAllAlgSeq                                         FATAL  Standard std::exception is caught
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAllAlgSeq                                         ERROR SG::ExcInvalidLink: Attempt to dereference invalid DataLink / ElementLink [1241842700/] (883500482)
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAlgEvtSeq                                         FATAL  Standard std::exception is caught
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthAlgEvtSeq                                         ERROR SG::ExcInvalidLink: Attempt to dereference invalid DataLink / ElementLink [1241842700/] (883500482)
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthMasterSeq                                         FATAL  Standard std::exception is caught
AODtoDAOD 17:37:56 2021-03-29 17:37:54,238 AthMasterSeq                                         ERROR SG::ExcInvalidLink: Attempt to dereference invalid DataLink / ElementLink [1241842700/] (883500482)

Hi @dzanzi,

Hmm - this is not an error I've seen before. Can you set the logging level to VERBOSE and then maybe we can see which chain this happens for. You might also have to tweak the default limits in the message service so that it doesn't hit the maximum number of messages.

Hi @jburr how can I set the verbose level only for the matching tool?

Hi @jburr , this chain gives the problem: HLT_mu6_LRT_idperf_l2lrt_L1MU6 Maybe there are others

added 92 commits

• 4fc1af36...5870cf98 - 90 commits from branch atlas:master
• c42f0106 - Merge remote-tracking branch 'upstream/master' into ATR-triggerListPHYS_dzanzi
• 0b14cce4 - veto LRT chains

Compare with previous version

marked this merge request as ready

changed the description

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys

This merge request affects 1 file:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys/share/PHYS.py

Adding @zmarshal ,@dshope as watchers

added Derivation master review-pending-level-1 labels

CI Result SUCCESS (hash 0b14cce4)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30911]

added 40 commits

• 0b14cce4...9a8109fd - 38 commits from branch atlas:master
• d1653bb6 - Merge remote-tracking branch 'upstream/master' into ATR-triggerListPHYS_dzanzi
• 4c8df707 - small fixes

Compare with previous version

resolved all threads

This merge request affects 1 package:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys

This merge request affects 1 file:

• PhysicsAnalysis/DerivationFramework/DerivationFrameworkPhys/share/PHYS.py

Adding @zmarshal ,@dshope as watchers

added Trigger label

CI Result SUCCESS (hash 4c8df707)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30943]

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

merged

mentioned in commit 4a20f5bd

added sweep:ignore label