Skip to content
Snippets Groups Projects

Update of CA configs for various tracking tools used for BPHY

Merged Thomas Strebler requested to merge tstreble/athena:BPHY_CA into 22.0
All threads resolved!

Continuation of updates of tracking CA configs, moving together the configs of the same C++ tools. This MR covers various tools relevant for BPHY derivations. Also adds a few configs relevant for V0Finder, based on InDetRecPostProcessing.py. Outputs of q445 tests unchanged as expected.

MR to 22.0 for keeping in sync CA configs there, will be followed up by a dedicated MR in master to plug this in derivation.

FYI @abarton @wleinone

Edited by Thomas Strebler

Merge request reports

Pipeline #4380832 passed

Pipeline passed for 872c69f0 on tstreble:BPHY_CA

Approval is optional

Merged by John Kenneth AndersJohn Kenneth Anders 2 years ago (Aug 19, 2022 2:53pm UTC)

Merge details

  • Changes merged into 22.0 with 12ec777a (commits were squashed).
  • Did not delete 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
  • added 1 commit

    • 872c69f0 - Fix default constructors for distance tool

    Compare with previous version

  • Thomas Strebler marked this merge request as ready

    marked this merge request as ready

  • Thomas Strebler changed the description

    changed the description

  • This merge request affects 5 packages:

    • InnerDetector/InDetConfig
    • InnerDetector/InDetRecTools/InDetConversionFinderTools
    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
    • Tracking/TrkConfig
    • Trigger/TrigMonitoring/TrigBphysMonitoring

    Affected files list will not be printed in this case

    Adding @goetz ,@bouhova ,@amorley ,@wwalko ,@sroe ,@abarton ,@tursom as watchers

  • Thomas Strebler resolved all threads

    resolved all threads

  • Thomas Strebler changed target branch from master to 22.0

    changed target branch from master to 22.0

  • Thomas Strebler added 22.0 label and removed master label

    added 22.0 label and removed master label

  • :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.

  • :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.

  • :white_check_mark: CI Result SUCCESS (hash 872c69f0)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :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: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :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
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: 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 57593]

  • This is CI from master. We need it from 22.0.

  • Jenkins please retry a build

  • This merge request affects 5 packages:

    • InnerDetector/InDetConfig
    • InnerDetector/InDetRecTools/InDetConversionFinderTools
    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkBPhys
    • Tracking/TrkConfig
    • Trigger/TrigMonitoring/TrigBphysMonitoring

    Affected files list will not be printed in this case

    Adding @goetz ,@bouhova ,@amorley ,@wwalko ,@sroe ,@abarton ,@tursom as watchers

  • :white_check_mark: CI Result SUCCESS (hash 872c69f0)

    Athena DetCommon
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    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 57619]

  • Approving - L1

  • mentioned in commit 12ec777a

  • Please register or sign in to reply
    Loading