Teach ISF new config test to run
Get ISF configuration to run a test without failing.
Many changes of private tools to public tools.
Add flag-based default decision for ConfigFlags.Sim.ISF.HITSMergingRequired (cc @jchapman )
Add Kernel_ATLFASTIIMTCfg
Get BaseSimulationSelector to accept an empty simulator, as needed for MT.
Numerous changes and suggestions are thanks to @bleban
Merge request reports
Activity
This merge request affects 9 packages:
- Simulation/G4Atlas/G4AtlasServices
- Simulation/ISF/ISF_Config
- Simulation/ISF/ISF_Core/ISF_Algorithms
- Simulation/ISF/ISF_Core/ISF_Services
- Simulation/ISF/ISF_Core/ISF_Tools
- Simulation/ISF/ISF_Geant4/ISF_Geant4CommonTools
- Simulation/ISF/ISF_Geant4/ISF_Geant4Services
- Simulation/ISF/ISF_Geant4/ISF_Geant4Tools
- Simulation/ISF/ISF_SimulationSelectors
added Simulation master review-pending-level-1 labels
CI Result SUCCESS (hash 6052a922)Athena AthSimulation AthGeneration AnalysisBase 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 19494]removed review-pending-level-1 label
added 372 commits
-
6052a922...04263fe9 - 358 commits from branch
atlas:master
- 53e7f324 - Link EvtStore as object, not string
- bf5fd461 - Enable cfg running
- e209aebc - Require FullChain for pileup stuff
- 723a8411 - Cfg fixes. Temp remove NumThreads == 0 req as it appears to cause errors.
- 9f0202ec - Undo NumThreads removal
- 958a3625 - Attempt to protect against empty m_simulator
- 277bbb21 - Fix constructer args, add printout
- 08069082 - Change to public tools
- d25bc5be - Remove debug printout
- a47c312f - Change more tools to public
- fe96579c - Duplicate code to cleanly separate, fix GeoIDSvc error
- 6fb3bcbf - Add HITS merging required flag
- ea85eb22 - Correction from Blaz
- 8f7c4a0c - Correct to public tools. Add missing Kernel_ATLFASTIIMTCfg
Toggle commit list-
6052a922...04263fe9 - 358 commits from branch
added 2 commits
added NewConfig label
- Resolved by Rupert Tombs
- Resolved by Rupert Tombs
- Resolved by Rupert Tombs
- Resolved by Rupert Tombs
- Resolved by Rupert Tombs
- Resolved by Rupert Tombs
added 2 commits
added 1 commit
- 7cb7a380 - Revert changes to AddNoise... because it is protected by tests. Reintroduce...