ATR-24438: switch to AllTag and AllProbe menu alignment groups
- move to menu alignment groups
AllTag
andAllProbe
for Electron, Photon, Muon, Bphys, Tau - change ChainMerging for chains with two signatures and two alignment groups in one of them; based on @hrussell's https://gitlab.cern.ch/hrussell/athena/-/commits/ATR24438_fix_doublemerge/ (enabled for all signatures in
AllTag
/AllProbe
alignment group) - some adjustments for CA and refactoring
tagging @astruebi , @lidiaz , @tamartin , @sshaw , @fpastore
Merge request reports
Activity
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 @martindl ,@ademaria ,@dbakshig ,@vmartin ,@okumura ,@carquin ,@jodafons ,@bernius ,@safarzad ,@shanisch ,@gipezzul ,@lidiaz ,@sutt ,@salderwe ,@iriu ,@fernando ,@hrussell ,@malconad ,@cjmeyer ,@eegidiop as watchers
CI Result SUCCESS (hash d9565526)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 65694]- Resolved by Antonia Strubig
- Resolved by Stefanie Morgenstern
No comments from me, but will mark as review-user-action-required while the open threads are being resolved
Albert (L1)added review-user-action-required label and removed review-pending-level-1 label
Thanks @shanisch !!
One thing to note - we will conflict against !60135 (merged) and I don't think there is anything really we can do to avoid this.
But after that, hopefully @lidiaz & @astruebi can help coordinate on the menu side to avoid further conflicts.
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 @martindl ,@ademaria ,@dbakshig ,@vmartin ,@okumura ,@carquin ,@jodafons ,@bernius ,@safarzad ,@shanisch ,@gipezzul ,@lidiaz ,@sutt ,@salderwe ,@iriu ,@fernando ,@hrussell ,@malconad ,@cjmeyer ,@eegidiop as watchers
added review-pending-level-1 label and removed review-user-action-required label
CI Result FAILURE (hash cac636b9)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 66220]added 18 commits
Toggle commit listThis 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 @martindl ,@ademaria ,@dbakshig ,@vmartin ,@okumura ,@carquin ,@jodafons ,@bernius ,@safarzad ,@shanisch ,@gipezzul ,@lidiaz ,@sutt ,@salderwe ,@iriu ,@fernando ,@hrussell ,@malconad ,@cjmeyer ,@eegidiop as watchers
CI Result SUCCESS (hash da456980)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 66237]