remove code and packages unneeded in Run 3. ATR-22797
further cleanup of the InDetTrigRecExample and of InDetTrigRecAlgs
Merge request reports
Activity
This merge request affects 12 packages:
- InnerDetector/InDetExample/InDetTrigRecExample
- InnerDetector/InDetTrigRecAlgs/InDetTrigPriVxFinder
- InnerDetector/InDetTrigRecAlgs/InDetTrigRawDataProvider
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackCollectionMerger
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackPRD_Association
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackSlimmer
- InnerDetector/InDetTrigRecAlgs/InDetTrigTruthAlgs
- InnerDetector/InDetTrigRecAlgs/InDetTrigVxSecondary
- InnerDetector/InDetTrigRecAlgs/SiTrigSPSeededTrackFinder
- InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
- InnerDetector/InDetTrigRecAlgs/TRT_TrigTrackSegmentsFinder
- InnerDetector/InDetTrigRecTools/InDetTrigToolInterfaces
Affected files list will not be printed in this case
Adding @sutt ,@goetz ,@calfayan ,@sroe ,@nagano ,@cgrefe as watchers
added InnerDetector Trigger master review-pending-level-1 labels
added full-build full-unit-tests labels
This merge request affects 12 packages:
- InnerDetector/InDetExample/InDetTrigRecExample
- InnerDetector/InDetTrigRecAlgs/InDetTrigPriVxFinder
- InnerDetector/InDetTrigRecAlgs/InDetTrigRawDataProvider
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackCollectionMerger
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackPRD_Association
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackSlimmer
- InnerDetector/InDetTrigRecAlgs/InDetTrigTruthAlgs
- InnerDetector/InDetTrigRecAlgs/InDetTrigVxSecondary
- InnerDetector/InDetTrigRecAlgs/SiTrigSPSeededTrackFinder
- InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
- InnerDetector/InDetTrigRecAlgs/TRT_TrigTrackSegmentsFinder
- InnerDetector/InDetTrigRecTools/InDetTrigToolInterfaces
Affected files list will not be printed in this case
Adding @sutt ,@goetz ,@calfayan ,@sroe ,@nagano ,@cgrefe as watchers
CI Result FAILURE (hash 3b94d04b)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 28517]Hi @jmasik,
can you please take look at the failing required tests? Reverting to review-user-action-required for now.
Cheers, Volker (L1)
added review-user-action-required label and removed review-pending-level-1 label
added 1 commit
- f5b3759e - remove imports from SiTrigSpacePointFormation
This merge request affects 12 packages:
- InnerDetector/InDetExample/InDetTrigRecExample
- InnerDetector/InDetTrigRecAlgs/InDetTrigPriVxFinder
- InnerDetector/InDetTrigRecAlgs/InDetTrigRawDataProvider
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackCollectionMerger
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackPRD_Association
- InnerDetector/InDetTrigRecAlgs/InDetTrigTrackSlimmer
- InnerDetector/InDetTrigRecAlgs/InDetTrigTruthAlgs
- InnerDetector/InDetTrigRecAlgs/InDetTrigVxSecondary
- InnerDetector/InDetTrigRecAlgs/SiTrigSPSeededTrackFinder
- InnerDetector/InDetTrigRecAlgs/SiTrigSpacePointFormation
- InnerDetector/InDetTrigRecAlgs/TRT_TrigTrackSegmentsFinder
- InnerDetector/InDetTrigRecTools/InDetTrigToolInterfaces
Affected files list will not be printed in this case
Adding @sutt ,@goetz ,@calfayan ,@sroe ,@nagano ,@cgrefe as watchers
added review-pending-level-1 label and removed review-user-action-required label
This compile warning ...
var/lib/jenkins/workspace/CI-MERGE-REQUEST-CC7/master/Trigger/TrigAlgorithms/TrigFastTrackFinder/python/TrigFastTrackFinder_Config.py:297:27: warning: E711 comparison to None should be 'if cond is None:' if conditionsTool == None:
was already fixed by Frank yesterday and can be ignored.
CI Result SUCCESS (hash f5b3759e)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis 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
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 28582]added review-approved label and removed review-pending-level-1 label
mentioned in commit 4b6c1c16
added sweep:ignore label