Add postInclude for adding length integrator user action to a job
The other part of moving from !41650 (closed) (along with !41741 (merged)). It's a bit less clear to me here that having this as a PostInclude is necessarily the best way to add this tool. However, it wasn't clear to me how else to do it simply such that this can be added to an arbitrary simulation job; the other UserActions seem to get configured and added based on what overall Simulation config is selected (e.g. ATLFastII, ISF, etc) which doesn't seem like what we want to do for this one.
Possibly the actual implementation of this as a PostInclude can also be improved (I struggled to get the private tools to work in a simpler manner, but that is likely just due to my own lack of familiarity with the new CA-based approach).
Merge request reports
Activity
added ITk Simulation labels
This merge request affects 1 package:
- Simulation/G4Utilities/G4UserActions
This merge request affects 2 files:
- Simulation/G4Utilities/G4UserActions/python/PostIncludes.py
- Simulation/G4Utilities/G4UserActions/python/init.py
Adding @jchapman as watcher
added master review-pending-level-1 labels
CI Result SUCCESS (hash b32600fd)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 30220]added review-approved label and removed review-pending-level-1 label
- Resolved by Tadej Novak
mentioned in commit 4fd4ef58
added sweep:ignore label