Skip to content
Snippets Groups Projects

Add missing required key:value pairs to /Digitization/Parameters metadata in CA-based configuration

All threads resolved!

This merge request ensures that the CA-based configuration correctly sets all key:value pairs in /Digitization/Parameters that are used in downstream configuration. It also adds auto-configuration for the corresponding ConfigFlags, so that they can be used in later steps.

Part of the work for ATLASSIM-5852

Edited by John Derek Chapman

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

    • 71f91f03 - Auto-configuration for Sim.PhysicsList and Digitization.DigiSteeringConf
    • b84cebcb - Now that flags.Digitization.DigiSteeringConf is auto-configured based on...

    Compare with previous version

  • Tadej Novak resolved all threads

    resolved all threads

  • John Derek Chapman marked this merge request as ready

    marked this merge request as ready

  • This merge request affects 4 packages:

    • Event/xAOD/xAODTruthCnv
    • Simulation/Digitization
    • Simulation/G4Atlas/G4AtlasApps
    • Tools/PyUtils

    Affected files list will not be printed in this case

    Adding @jchapman ,@akraszna as watchers

  • :white_check_mark: CI Result SUCCESS (hash b84cebcb)

    Athena DetCommon
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    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: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53160]

  • CI and code look fine to me. Approving. L1

  • Julien Maurer mentioned in commit 808f212f

    mentioned in commit 808f212f

  • merged

  • Please register or sign in to reply
    Loading