New magnetic field switch class
Addition of a new class for tailored magnetic field switch-off in ATLAS simulation in order to save CPU power. This switch is operating in the area inside the EM barrel calorimeter region, where the magnetic field is low anyways, and for all particles except of muons.
The modifications include:
- Addition of a new class
SwitchingFieldManager
, inheriting fromG4FieldManager
. - Modification of the
DetectorFieldManagerTool.cxx
to use the alternativeG4FieldManager
when requested. Here there was a previous attempt to achieve the same effect by theTightMuonElseNoFieldManager
and it has been commented out. - Modification of the
DetectorFieldManagerTool
to be able to attach aG4FieldManager
to aG4VPhysicalVolume
as well, instead ofG4LogicalVolume
only as it was before. - Modification in
G4FieldConfig.py
to attach theSwitchingFieldManager
to theLArBarrel
physical volume. Μodifications have not been applied inG4FieldConfigNew.py
. (maybe we should?)
The switch can be enabled with the pre-include:
simFlags.MuonFieldOnlyInCalo=True;simFlags.MuonFieldOnlyInCalo()
. Without this flag the developments should not affect the simulation jobs.
The idea and the development progress is documented in ATLASSIM-4749.
Merge request reports
Activity
This merge request affects 2 packages:
- Simulation/G4Atlas/G4AtlasServices
- Simulation/G4Atlas/G4AtlasTools
Affected files list will not be printed in this case
Adding @jchapman as watcher
added Simulation master review-pending-level-1 labels
CI Result SUCCESS (hash a7d96a9c)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 35849]- Resolved by Vangelis Kourlitis
- Resolved by Vangelis Kourlitis
- Resolved by Vangelis Kourlitis
- Resolved by Vangelis Kourlitis
- Resolved by Clara Nellist
added review-user-action-required label and removed review-pending-level-1 label
This merge request affects 2 packages:
- Simulation/G4Atlas/G4AtlasServices
- Simulation/G4Atlas/G4AtlasTools
Affected files list will not be printed in this case
Adding @jchapman as watcher
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash bfc2b786)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 35880]Changing label to review-user-action-required until the discussions are resolved. Please change back when ready. Clara (L1)
added review-user-action-required label and removed review-pending-level-1 label
added 85 commits
-
bfc2b786...d4914b40 - 83 commits from branch
atlas:master
- ce845b26 - remove cout commented-out lines
- 49a9cb62 - Merge branch 'master' of https://gitlab.cern.ch:8443/atlas/athena
-
bfc2b786...d4914b40 - 83 commits from branch
removed review-user-action-required label
added review-pending-level-1 review-user-action-required labels
removed review-user-action-required label