sweep missing infrastructure changes from 21.2 to master
I hope I got them all, but these seem to be the remaining commits that I never swept to master.
@tadej, If I missed one, let us know.
- caf47153 update to use the algorithm meta-configuration for jet selection
- bc4c7808 make copies of python lists from meta-configuration
- 9ef420f0 switch selection accounting to use algorithm meta-configuration
- 46d4457e switch to using meta-configuration for selection accounting
- dd77137d move call to xAOD::LoadDictionaries() earlier
Merge request reports
Activity
This merge request affects 5 packages:
- PhysicsAnalysis/Algorithms/EgammaAnalysisAlgorithms
- PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms
- PhysicsAnalysis/Algorithms/MuonAnalysisAlgorithms
- PhysicsAnalysis/Algorithms/TauAnalysisAlgorithms
- PhysicsAnalysis/D3PDTools/EventLoop
Affected files list will not be printed in this case
CI Result SUCCESS (hash 450ddb46)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 40642]Hi @krumnack, thanks for the sweep! I think we got the main ones. I'll let you know if I spot anything else missing.
added review-approved label and removed review-pending-level-1 label
@krumnack , ironically, you (or somebody else) will need to analysis-approve this one too
added analysis-review-approved label and removed analysis-review-required label
mentioned in commit 6d81b516
added sweep:ignore label