Skip to content
Snippets Groups Projects

New magnetic field switch class

Merged Vangelis Kourlitis requested to merge ekourlit/athena:master into master

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:

  1. Addition of a new class SwitchingFieldManager, inheriting from G4FieldManager.
  2. Modification of the DetectorFieldManagerTool.cxx to use the alternative G4FieldManager when requested. Here there was a previous attempt to achieve the same effect by the TightMuonElseNoFieldManager and it has been commented out.
  3. Modification of the DetectorFieldManagerTool to be able to attach a G4FieldManager to a G4VPhysicalVolume as well, instead of G4LogicalVolume only as it was before.
  4. Modification in G4FieldConfig.py to attach the SwitchingFieldManager to the LArBarrel physical volume. Μodifications have not been applied in G4FieldConfigNew.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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • 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

  • :white_check_mark: CI Result SUCCESS (hash bfc2b786)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: 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 85 commits

    Compare with previous version

  • Vangelis Kourlitis resolved all threads

    resolved all threads

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading