Skip to content
Snippets Groups Projects

Unified new config test for InnerDetector and Muon digitization

Merged Rupert Tombs requested to merge rtombs/athena:DigitizationConfigTest into master

Test BCM, SCT, MDT, TGC, RPC, and CSC configuration modules in execution on default test inputs.

Executed as

python Simulation/Digitization/test/DigitizationConfigNew_test.py

Fix the BCM test to use BCM, and tidy it up.

Edited by Rupert Tombs

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
  • Tadej Novak
  • Tadej Novak
  • Rupert Tombs added 33 commits

    added 33 commits

    • b4a2a470...050d06fb - 32 commits from branch atlas:master
    • 7bdbbfa4 - Add combined test for InnerDetector and Muon new config modules. Fix BCM test to include BCM.

    Compare with previous version

  • Rupert Tombs unmarked as a Work In Progress

    unmarked as a Work In Progress

  • This merge request affects 6 packages:

    • InnerDetector/InDetDigitization/BCM_Digitization
    • MuonSpectrometer/MuonDigitization/CSC_Digitization
    • MuonSpectrometer/MuonDigitization/MDT_Digitization
    • MuonSpectrometer/MuonDigitization/RPC_Digitization
    • MuonSpectrometer/MuonDigitization/TGC_Digitization
    • Simulation/Digitization

    Adding @jchapman ,@rosati ,@vpascuzz ,@wleight as watchers

  • :white_check_mark: CI Result SUCCESS

    Athena AthSimulation
    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 at NICOS MR-22597-2019-04-11-15-53
    :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 36908]

  • Rupert Tombs added 13 commits

    added 13 commits

    • 7bdbbfa4...52720e55 - 12 commits from branch atlas:master
    • 2cb5e7f1 - Add combined test for InnerDetector and Muon new config modules. Fix BCM test to include BCM.

    Compare with previous version

  • This merge request affects 6 packages:

    • InnerDetector/InDetDigitization/BCM_Digitization
    • MuonSpectrometer/MuonDigitization/CSC_Digitization
    • MuonSpectrometer/MuonDigitization/MDT_Digitization
    • MuonSpectrometer/MuonDigitization/RPC_Digitization
    • MuonSpectrometer/MuonDigitization/TGC_Digitization
    • Simulation/Digitization

    Adding @jchapman ,@rosati ,@vpascuzz ,@wleight as watchers

  • :white_check_mark: CI Result SUCCESS

    Athena AthSimulation
    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 at NICOS MR-22597-2019-04-11-17-25
    :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 36915]

  • :white_check_mark: CI Result SUCCESS

    Athena AthSimulation
    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 at NICOS MR-22597-2019-04-11-20-42
    :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 36927]

  • Rupert Tombs resolved all discussions

    resolved all discussions

  • Looks good, due to extent of the changes passing to L2.

    Cheers, Tomas (L1)

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