add a addPrivateToolInArray function to DualUseConfig.py
Unfortunately I don't have a really good way to test if it works, but the underlying C++ is tested, and this is hopefully a close enough duplication of what's happening for non-array tools that this is good enough.
Originally I tried to do something more involved, but that became sort of tedious given the logic of the python class. Maybe at some point I will come back and restructure the python class somewhat. Hopefully this is still workable as is.
cc @delsart @khoo @wbalunas (who asked for dual-use ToolHandleArray
support for the jet group)
cc @akraszna (who originally wrote the python AnaAlgorithmConfig
)
Merge request reports
Activity
This merge request affects 1 package:
- PhysicsAnalysis/D3PDTools/AnaAlgorithm
This merge request affects 2 files:
- PhysicsAnalysis/D3PDTools/AnaAlgorithm/python/AnaAlgorithmConfig.py
- PhysicsAnalysis/D3PDTools/AnaAlgorithm/python/DualUseConfig.py
Adding @krumnack as watcher
added Analysis master review-pending-level-1 labels
CI Result SUCCESS (hash d6b07dae)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 32061]added review-approved label and removed review-pending-level-1 label
mentioned in commit 78402347
added sweep:ignore label