Skip to content
Snippets Groups Projects

ATR-22298 Isolated, high pT trigger, CPU Reduction Changes

Merged Ismet Siral requested to merge isiral/athena:UnconvIsolatedTrackTrigger into master

I am introducing a new pT cut for the Isolated Track Trigger, that introduces a significant CPU reduction for the trigger. In addition, I have changed the naming scheme of cumulative (icummedium) isolation to aggravated (iaggrmeduim) as the previous acronym icum was unintentionally vulgar.

@ljeanty @shanisch @sutt @nagano

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
  • Thanks @isiral, I have only a few minor comments. Tom - L1 shifter

  • Ismet Siral added 696 commits

    added 696 commits

    Compare with previous version

  • This merge request affects 8 files:

    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/python/TrigIsoHPtTrackTriggerHypoTool.py
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoAlgMT.cxx
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoTool.cxx
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoTool.h
    • Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref
    • Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/SignatureDicts.py

    Adding @sutt ,@ademaria ,@okumura ,@carquin ,@bernius ,@hrussell ,@ckaldero ,@malconad ,@jpanduro as watchers

  • Ismet Siral resolved all threads

    resolved all threads

  • :negative_squared_cross_mark: CI Result FAILURE (hash 3f6553d2)

    Athena AthSimulation AthGeneration AnalysisBase
    externals :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:
    make :warning: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :o: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :warning: Athena: number of compilation errors 0, warnings 1
    :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23529]

  • Ismet Siral added 99 commits

    added 99 commits

    Compare with previous version

  • This merge request affects 8 files:

    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/python/TrigIsoHPtTrackTriggerHypoTool.py
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoAlgMT.cxx
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoTool.cxx
    • Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoTool.h
    • Trigger/TrigValidation/TrigAnalysisTest/share/ref_RDOtoRDOTrig_v1Dev_build.ref
    • Trigger/TrigValidation/TriggerTest/share/ref_data_v1Dev_build.ref
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/LS2_v1.py
    • Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Menu/SignatureDicts.py

    Adding @sutt ,@ademaria ,@okumura ,@carquin ,@bernius ,@hrussell ,@ckaldero ,@malconad ,@jpanduro as watchers

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