Skip to content
Snippets Groups Projects

Switch to CA-config for FastCaloSim in Geant4

This MR switches the FastCaloSim implementation within Geant4 to use the CA config. Many thanks to @jchapman for the help.

No output differences with 20 ttbar events compared to CG config

Edited by Joshua Falco Beirer

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 27 commits

    • 55a99725...008d4d9a - 17 earlier commits
    • 3ffaa600 - Update TrigVrtSecInclusiveCfg to use central vertex configs
    • fb612a5b - Trackless variables SiHit information
    • 8eff46cc - Simplify BTAG_TrkVKalVrtFitterCfg
    • ffb84f57 - Remove egammaCalo Tool Factories
    • 8600ef90 - egammaTrackTools rm legacy config
    • ff433f0a - egammaMVACalib rm legacy
    • 080993b4 - Remove egamma legacy GSF config
    • 5fb8d67e - Adding hough reweighting for MM
    • b96cbdbc - Implements electron removed alternative TauJet reconstruction
    • e4e8c45a - Migrate further config to CA

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Joshua Falco Beirer added 117 commits

    added 117 commits

    Compare with previous version

  • Currently, many differences with respect to CG configuraiton observed, also in ID branches. Attaching conf-tool output for reference. Might be result of different ordering of alignment algs? conf_tool.log

  • John Derek Chapman mentioned in merge request !68376 (merged)

    mentioned in merge request !68376 (merged)

  • John Derek Chapman mentioned in merge request !68660 (closed)

    mentioned in merge request !68660 (closed)

  • Joshua Falco Beirer added 2436 commits

    added 2436 commits

    • 9c69af1e...461e2543 - 2428 commits from branch atlas:main
    • b4950b29 - Switch to CA-config for FCS in G4 test
    • a2cfa990 - Migrate further config to CA
    • 409c6462 - Fixed config
    • b1a38472 - Revised AtlasG4_FastCaloSim ART jobs - test_AtlasG4_FastCaloSim_MT.sh is only there temporarily
    • 0d69e6ea - AtlasG4 FastCaloSim running in AthenaMT (WIP)
    • 0037a954 - Revised AtlasG4_FastCaloSim ART jobs - test_AtlasG4_FastCaloSim_MT.sh is only there temporarily
    • 1e2855c7 - Adding DeadMaterialShowers for FCS in G4 jobs - need to check that this makes sense
    • 66915710 - Force the ordering of inputs to the CollectionMerger

    Compare with previous version

  • Jenkins please retry a build

  • Joshua Falco Beirer changed the description

    changed the description

  • added 1 commit

    Compare with previous version

  • This merge request affects 10 packages:

    • Simulation/BeamEffects
    • Simulation/G4Atlas/G4AtlasAlg
    • Simulation/G4Atlas/G4AtlasTools
    • Simulation/G4Utilities/G4FastSimulation
    • Simulation/ISF/ISF_Core/ISF_Algorithms
    • Simulation/ISF/ISF_FastCaloSim/ISF_FastCaloSimParametrization
    • Simulation/SimuJobTransforms
    • Simulation/SimulationConfig
    • Simulation/Tests/SimCoreTestsMT
    • TileCalorimeter/TileConditions

    Affected files list will not be printed in this case

    Adding @iouri ,@solodkov ,@harkusha ,@zhangr ,@schaarsc ,@pavol ,@tadej ,@jchapman ,@lmijovic ,@jbeirer as watchers

  • Joshua Falco Beirer resolved all threads

    resolved all threads

  • :white_check_mark: CI Result SUCCESS (hash 66915710)

    Athena AthSimulation AnalysisBase AthAnalysis
    externals :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:
    make :warning: :warning: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

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

  • added 1 commit

    Compare with previous version

  • Joshua Falco Beirer marked this merge request as ready

    marked this merge request as ready

  • Joshua Falco Beirer changed the description

    changed the description

  • Jenkins please retry a build

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