add AnaReentrantAlgorithm
Mostly a copy-and-paste of AnaAlgorithm, with cleanup and updates to inherit from AthReentrantAlgorithm.
Probably the main thing to look for for the reviewer are copy-and-paste errors.
Not sure if it was @tadej, @jburr, or @khoo who kept asking for this, but here it is. Please note that the systematics handles will need updates to work with reentrant algorithms (to say nothing of the ISystematicsTool
).
Merge request reports
Activity
This merge request affects 3 packages:
- PhysicsAnalysis/D3PDTools/AnaAlgorithm
- PhysicsAnalysis/D3PDTools/EventLoop
- PhysicsAnalysis/D3PDTools/EventLoopTest
Affected files list will not be printed in this case
Adding @krumnack as watcher
added Analysis master review-pending-level-1 labels
CI Result SUCCESS (hash 37e5611d)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 1
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 26835]- Resolved by Nils Erik Krumnack
added review-user-action-required label and removed review-pending-level-1 label
added 108 commits
-
37e5611d...8d4655e9 - 107 commits from branch
atlas:master
- d965898b - add AnaReentrantAlgorithm
-
37e5611d...8d4655e9 - 107 commits from branch
This merge request affects 3 packages:
- PhysicsAnalysis/D3PDTools/AnaAlgorithm
- PhysicsAnalysis/D3PDTools/EventLoop
- PhysicsAnalysis/D3PDTools/EventLoopTest
Affected files list will not be printed in this case
Adding @krumnack as watcher
added review-pending-level-1 label and removed review-user-action-required label
CI Result FAILURE (hash d965898b)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 26940]added review-approved label and removed review-pending-level-1 label
mentioned in commit 65ce9681
added sweep:ignore label
mentioned in merge request !42111 (merged)