ATR-22298 Isolated, high pT trigger, CPU Reduction Changes
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.
Merge request reports
Activity
This merge request affects 6 files:
- Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/python/TrigIsoHPtTrackTriggerHypoTool.py
- Trigger/TrigHypothesis/TrigLongLivedParticlesHypo/src/TrigIsoHPtTrackTriggerHypoAlgMT.cxx
- 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
added Trigger TriggerMenu master review-pending-level-1 labels
CI Result SUCCESS (hash 056eb72c)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23067]- Resolved by Ismet Siral
- Resolved by Ismet Siral
- Resolved by Ismet Siral
- Resolved by Ismet Siral
added review-user-action-required label and removed review-pending-level-1 label
added 696 commits
-
056eb72c...763fcf3e - 690 commits from branch
atlas:master
- c63b7f75 - Changes in the unit system as requested
- 96881e6d - Merge branch 'master' into UnconvIsolatedTrackTrigger
- 81a5c537 - Merge branch 'master' into UnconvIsolatedTrackTrigger
- 6ab22a3d - merge conflicts
- 504b5cc1 - Finnish merge
- 3f6553d2 - Update References
Toggle commit list-
056eb72c...763fcf3e - 690 commits from branch
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
added changes-trigger-counts review-pending-level-1 labels and removed review-user-action-required label
CI Result FAILURE (hash 3f6553d2)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 1
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 23529]- Resolved by Ismet Siral
Failed TriggerMenuMT_flake8_ctest and there are merge conflicts, can you check it?
added review-user-action-required label and removed review-pending-level-1 label
added 99 commits
-
3f6553d2...3f933dae - 96 commits from branch
atlas:master
- 6737b53d - Merge conflict fixed
- 3f86b746 - Fixing un-intentional changes
- 8206ae32 - Updateding refferences
Toggle commit list-
3f6553d2...3f933dae - 96 commits from branch
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
added review-pending-level-1 label and removed review-user-action-required label