Skip to content
Snippets Groups Projects

move to ca for id art tests

Merged Ke Li requested to merge keli/athena:useca into 23.0

Update all the ID ART tests to CA for both rel. 23 and 24. Use Zprime to for IDTIDE tests. The RDO and configurations are following MC23a which is validated. The reference for rel. 23 and 24 are also updated. Everything looks fine from several local tests. This MR should also go into master.

Ping @tstreble, @gfacini , @cgrefe , and the shifter in this week and next @honguyen and @bpascual, also ping @srettie and @nstyles

Edited by Ke Li

Merge request reports

Pipeline #5890142 passed

Pipeline passed for 3ffbe07d on keli:useca

Approval is optional

Merged by Walter LamplWalter Lampl 1 year ago (Jul 14, 2023 8:31am UTC)

Merge details

  • Changes merged into 23.0 with 54d349be (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thomas Strebler
  • Ke Li added 4 commits

    added 4 commits

    • bcd4003f - add the config for ID pixel and sct hit analysis
    • 4082305a - add the config for ID pixel and sct hit analysis
    • f0ea653e - update the simreco test to ca
    • 15a01943 - Merge remote-tracking branch 'origin/useca' into useca

    Compare with previous version

  • removed master label

  • Ke Li added 1 commit

    added 1 commit

    • 2cec0cca - update the single partice and simreco test to ca

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    • c8f81727 - upate the input to mc23a and add zprime test for idtide

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    • 7f457240 - disable the vertextruthmatch tool when running idtide

    Compare with previous version

  • Ke Li added 1 commit

    added 1 commit

    Compare with previous version

  • Ke Li marked this merge request as ready

    marked this merge request as ready

  • Ke Li added master label

    added master label

  • Ke Li resolved all threads

    resolved all threads

  • This merge request affects 2 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • Simulation/Tools/HitAnalysis

    Affected files list will not be printed in this case

    Adding @goetz ,@jchapman ,@lgagnon ,@sroe ,@keli as watchers

  • :pencil: Build area was cleaned as per request posted in the DB. The full software build will be performed

  • :pencil: There were multiple CI triggers for this MR and commit. The system ignored duplicates but the GitLab pipeline status may incorrectly show the job as failed. Once the remaining job finished running, the CI results will be posted as usual.

  • Ke Li changed the description

    changed the description

  • added InnerDetector NewConfig Simulation labels and removed master label

  • Fixed labels - MRs for 23.0 are swept to master by default.

  • John Derek Chapman
  • Ke Li resolved all threads

    resolved all threads

  • Thomas Strebler
  • John Derek Chapman
  • :white_check_mark: CI Result SUCCESS (hash 94fecb7e)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 72702]

  • Changes and CI look fine from L1 point of view but please address the open threads.

    Kira (L1)

  • Ke Li added 1 commit

    added 1 commit

    • 3ffbe07d - update the configuration based mc23a compaign

    Compare with previous version

  • This merge request affects 2 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • Simulation/Tools/HitAnalysis

    Affected files list will not be printed in this case

    Adding @goetz ,@jchapman ,@lgagnon ,@sroe ,@keli as watchers

  • :white_check_mark: CI Result SUCCESS (hash 3ffbe07d)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 74490]

  • Ke Li resolved all threads

    resolved all threads

  • Ke Li resolved all threads

    resolved all threads

  • CI clean, changes look OK from an L1 point of view. Approving - L1

  • Walter Lampl mentioned in commit 54d349be

    mentioned in commit 54d349be

  • merged

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