Skip to content
Snippets Groups Projects

Sim_tf - Toggle container output depending on whether AMDB or PhaseII style geometry is loaded

Merged Johannes Junggeburth requested to merge jojungge/athena:MuonSimContainers into main
  • Automatically toggle the name of the simulation containers depending on whether the R4 style or AMDB style geometry is parsed to the job
  • Declare output containers in order to run the geometry
  • Do not schedule the Geometry translation in the simulation step

A HITS file with the new geometry file can be produced via:

GeoTag="ATLAS-R3S-2021-03-02-00"
GeoPath="${PWD}/Geometry/${GeoTag}.db"

export ATHENA_PROC_NUMBER=1
export ATHENA_CORE_NUMBER=1
Sim_tf.py \
    --CA \
    --multithreaded True \
    --geometrySQLite True \
    --inputEVNTFile /cvmfs/atlas-nightlies.cern.ch/repo/data/data-art/MuonRecRTT/EVGEN_ParticleGun_FourMuon_Pt10to500.root \
    --conditionsTag 'default:OFLCOND-MC21-SDR-RUN3-05' \
    --simulator 'FullG4MT_QS' \
    --preExec "EVNTtoHITS:from AtlasGeoModel import CommonGeoDB;CommonGeoDB.SetupLocalSqliteGeometryDb('${GeoPath}','${GeoTag}');flags.Scheduler.ShowDataDeps=True;" \
    --postInclude 'PyJobTransforms.TransformUtils.UseFrontier' \
    --preInclude 'EVNTtoHITS:Campaigns.MC21Simulation' \
    --postExec 'EVNTtoHITS:DetDescCnvSvc=cfg.getService("DetDescrCnvSvc");DetDescCnvSvc.IdDictFromRDB=False;DetDescCnvSvc.MuonIDFileName="IdDictParser/IdDictMuonSpectrometer_R.09.03.xml";cfg.printConfig(withDetails=True, summariseProps=True);flags.dump(evaluate = True);' \
    --geometryVersion "default:${GeoTag}" \
    --outputHITSFile SimTf.pool.root \
    --maxEvents '10' \
    --imf False \
    --detectors MDT RPC TGC MM sTGC

Tagging @boudreau, @tsulaia, @emoyse, @jchapman

Edited by Johannes Junggeburth

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
  • Johannes Junggeburth resolved all threads

    resolved all threads

  • added 1 commit

    • 7d2b7165 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • This merge request affects 4 packages:

    • MuonSpectrometer/MuonG4/MuonG4SD
    • MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4
    • MuonSpectrometer/MuonPhaseII/MuonG4/MuonSensitiveDetectorsR4
    • Simulation/SimuJobTransforms

    Affected files list will not be printed in this case

    Adding @stavrop ,@pscholer ,@kluit ,@apsallid ,@jchapman ,@rosati ,@jojungge as watchers

  • :x: CI Result FAILURE (hash 7d2b7165)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    tests :o: :o:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 5022]

  • added 9 commits

    • 7d2b7165...eb5391a4 - 4 commits from branch atlas:main
    • a27f37c4 - Add switch to toggle between R4& R3 output
    • 391b0413 - Aux container name
    • cb9e23d1 - Apply 1 suggestion(s) to 1 file(s)
    • e16252d8 - Found switch for the Sim_tf output as well
    • 3cde2db4 - Inject the containers in Sim_tf as well. Fix data dependencies

    Compare with previous version

  • This merge request affects 7 packages:

    • MuonSpectrometer/MuonConfig
    • MuonSpectrometer/MuonG4/MuonG4SD
    • MuonSpectrometer/MuonPhaseII/MuonConditions/MuonCondAlgR4
    • MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4
    • MuonSpectrometer/MuonPhaseII/MuonG4/MuonSensitiveDetectorsR4
    • Simulation/ISF/ISF_Config
    • Simulation/SimuJobTransforms

    Affected files list will not be printed in this case

    Adding @rosati ,@apsallid ,@pscholer ,@jchapman ,@stavrop ,@jojungge ,@kluit as watchers

  • Johannes Junggeburth changed the description

    changed the description

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