Skip to content
Snippets Groups Projects

Trigger CA migration for combined muon reco sequence

Merged Savanna Shaw requested to merge sshaw/athena:trigmuonca-idprec into main

Moving the combined muon trigger reconstruction sequence to be entirely CA based and then wrapping it with the algorithmCAToGlobalWrapper in the menu sequences (ATR-25086):

  • Switched the precision ID tracking to the CA based version and added an option to the ID configuration to pass the correct RoI collection so that we get the RoIs each step is actually running in (and updated the default for LRT to a collection that exists)
  • Cloned the flags needed for the ID tracking
  • Some minor updates to view data verifiers
  • Remove the wrappers from the individual algorithm CA pieces

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
  • Savanna Shaw added 442 commits

    added 442 commits

    • 78ae6f4c...befe908f - 436 commits from branch atlas:main
    • a011c8a8 - Muon trigger CA migration for precision combined muons
    • b2247a8c - Add cloneAndReplace for ID flags, update VDV, fix typo in variable names
    • bf57091e - Fix sequence definition
    • b3931c2e - Fixes after rebasing to main
    • 23b79382 - Update RoIs used for ID tracks
    • 29409c1c - Revert RoI passing to IDConfig

    Compare with previous version

  • Savanna Shaw added 1 commit

    added 1 commit

    • 83d5a37b - too hasty and accidentally deleted a return

    Compare with previous version

  • Savanna Shaw resolved all threads

    resolved all threads

  • Savanna Shaw marked this merge request as ready

    marked this merge request as ready

  • Author Developer

    Jenkins please retry a build

  • This merge request affects 1 package:

    • Trigger/TriggerCommon/TriggerMenuMT

    This merge request affects 2 files:

    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonMenuSequences.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonRecoSequences.py

    Adding @sutt ,@lidiaz as watchers

  • :pencil: :scissors: The system selected 54 tests to probe the Athena changeset (out of 65 available tests). Link to tests selection rules

  • :white_check_mark: CI Result SUCCESS (hash 83d5a37b)

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

  • Savanna Shaw added 1 commit

    added 1 commit

    • 27ba0508 - remove unused imports/variables

    Compare with previous version

  • This merge request affects 1 package:

    • Trigger/TriggerCommon/TriggerMenuMT

    This merge request affects 2 files:

    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonMenuSequences.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonRecoSequences.py

    Adding @sutt ,@lidiaz as watchers

  • :pencil: :scissors: The system selected 54 tests to probe the Athena changeset (out of 65 available tests). Link to tests selection rules

  • :white_check_mark: CI Result SUCCESS (hash 27ba0508)

    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 79330]

  • Jiri Masik
  • Jiri Masik
  • Jiri Masik
  • Savanna Shaw added 1 commit

    added 1 commit

    • b92a6f9d - Remove uneeded ID containers from VDVs

    Compare with previous version

  • Savanna Shaw resolved all threads

    resolved all threads

  • This merge request affects 1 package:

    • Trigger/TriggerCommon/TriggerMenuMT

    This merge request affects 2 files:

    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonMenuSequences.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLT/Muon/MuonRecoSequences.py

    Adding @sutt ,@lidiaz as watchers

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