Photon ringer chain

Inclusion of Photon Ringer chains as described in ATR-24384.

Pinging @jodafons @cjmeyer @fernando @eegidiop @safarzad

added Egamma Trigger master labels

assigned to @jlieberm

Hi @jlieberm,

can you please add the atlasbot as a member of your athena fork to avoid issues with running the CI? The details are here.

Jakub (L1 shifter)

Hi @jkremer ,

Done! Should I try to solve the conflicts or the CI will run again?

Thanks!

added 316 commits

• d506627b...1c829886 - 312 commits from branch atlas:master
• 10aca98c - add and config photon ringer chain
• 30d94b24 - update v1 dev ref file
• e2d684f3 - update decodeBS ref file
• a41e7c97 - fix conflicts

Compare with previous version

added 1 commit

• 432781aa - update ref files

Compare with previous version

marked this merge request as ready

WARNING: big files (>100K) are found in the changeset

160K in file Trigger/TrigValidation/TrigP1Test/share/ref_v1Dev_decodeBS_build.ref

256K in file Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref

112K in file Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Menu/Dev_pp_run3_v1.py

This merge request affects 4 packages:

• Trigger/TrigHypothesis/TrigEgammaHypo
• Trigger/TrigValidation/TrigAnalysisTest
• Trigger/TrigValidation/TrigP1Test
• Trigger/TriggerCommon/TriggerMenuMT

Affected files list will not be printed in this case

Adding @sutt ,@dzanzi ,@dbakshig ,@salderwe ,@vmartin ,@okumura ,@jodafons ,@fernando ,@bernius ,@hrussell ,@cjmeyer as watchers

added TriggerMenu changes-trigger-counts review-pending-level-1 labels

CI Result SUCCESS (hash 432781aa)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional 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
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 46368]

The changes look fine and there are no CI issues. Approving.

Jason (L1)

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

resolved all threads

merged

mentioned in commit 0eea7793

added sweep:ignore label

mentioned in merge request !50284 (merged)

mentioned in merge request !50325 (merged)