PerfMonComps: Update PerfMonMT CA configuration
This MR should fix the failing CA-based simulation ART tests in the latest nightly. The problem originated from two MRs that did play nicely with each other: !47011 (merged) and !47014 (merged). Originally we were inserting PerfMonMTAlg
to the standard AthAlgSeq
. After the former, this algorithm is now being added to AthBeginSeq
but by default a bare ComponentAccumulator
doesn't know about this. So, things needed to be adjusted a little bit to accomodate that. Let me cc @jchapman and @tadej so that they're aware.
Also, the legacy configuration allows one to insert an algorithm to anywhere in a sequence (by index). Admittedly, this can be a little fiddly but it was nice to have. This doesn't seem to be possible in the new configuration from a quick glance. Maybe @wlampl has an idea on that.
Merge request reports
Activity
added Core master review-pending-level-1 labels
CI Result SUCCESS (hash 8d00eac9)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 40115]This should fix ATLASSIM-5436.
added review-approved label and removed review-pending-level-1 label
mentioned in commit 0a99c3a5
added sweep:ignore label