Increase flexibility of CA sequences merging
In HLT use-case we have sometimes situation when the signatures code need to provide algorithms that need to be inserted in HLT CF and some other in the top-level sequences structure. This MR enables it. Simple the code needs to generate a CA with multiple sequences. The main one is used in CF while all others are eventually added to the tree somewhere else.
There is an example (that will be uncommented in the follow-up MR) of how it will be used in a particular case of providing BCIDAvg to all HLT calo algorithms.
Also, improved a bit of messaging and suppressed unmerged CA deletion errors when there is an exception. Tagging @wlampl
Merge request reports
Activity
added Core Trigger master review-pending-level-1 labels
CI Result FAILURE (hash 41b91b53)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 27750] CI Result SUCCESS (hash 41b91b53)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 27798]added review-approved label
removed review-pending-level-1 label
Hi @tbold , please fix merge conflicts. Thanks!
added review-user-action-required label
added 131 commits
-
41b91b53...be1d99b4 - 130 commits from branch
atlas:master
- d6118d9e - Merge remote-tracking branch 'upstream/master' into support-many-ca-per-step
-
41b91b53...be1d99b4 - 130 commits from branch
added review-pending-level-1 label and removed review-approved review-user-action-required labels
CI Result SUCCESS (hash d6118d9e)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 27883]added review-approved label and removed review-pending-level-1 label
mentioned in commit 6dd5de35
added sweep:ignore label
mentioned in merge request !40591 (merged)
added NewConfig label