WIP: jet hypo - generalisation of jet hypo Filters
Filters are used to reduce the number of jets from an input jet vector. Until now the filters have been confined to those that use a sequence of capacity = 1 Conditions to make the selection. Dummy filtering (ie no selection) was arranged by having the dummy filter have no conditions.
A new use case has come up where filtering is needed, but cannot be done as described above. A new oure interface is now used ot allow different filtering implementations. Dummy filtering is now down with a dedicated named implementation class.
Filtering currently is done both on the input vector beore the hypo code proper is run, and as a jet selector to Fast Reducer leaf Conditions
Other changes: Repeated Condition can now invert its decision base on a configurable boolean flag. ConditionFilter uses Repeated Condition, rather than CompoundCondition directly
New interfaces: IHypoJetVectorFilter.h ITrigHypoJetVectorFilterConfig.h
Pass through filtering PassThroughFilter.h, cxx TrigJetPassThroughFilterConfig.h, cxx
New filter configurers TrigJetConditionFilterConfig.cxx, h TrigJetPassThroughFilterConfig.cxx, h TrigJetRangeFilterConfig.cxx, h TrigJetHypoToolHelperNoGrouper.h, cxx
Condition inversion: RepeatedCondition.cxx, h TrigJetConditionConfig_repeated.cxx, h ITrigJetRepeatedConditionConfig.h
Move to use RepeatedCondition ConditionFilter.cxx, h TrigJetHypoToolConfig_fastreduction.cxx
Merge request reports
Activity
added JetHLTHypo master labels
added 154 commits
-
e2bf52bd...72b75ee5 - 153 commits from branch
atlas:master
- d3f0ca6c - Merge remote-tracking branch 'upstream/master' into master-22914-1, fix coflict.
-
e2bf52bd...72b75ee5 - 153 commits from branch
This merge request affects 1 package:
- Trigger/TrigHypothesis/TrigHLTJetHypo
This merge request affects 21 files:
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/ConditionFilter.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/ConditionFilter.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/IHypoJetVectorFilter.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/ITrigHypoJetVectorFilterConfig.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/ITrigJetRepeatedConditionConfig.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/PassThroughFilter.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/PassThroughFilter.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/RepeatedCondition.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/RepeatedCondition.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetConditionConfig_repeated.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetConditionConfig_repeated.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetConditionFilterConfig.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetConditionFilterConfig.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetHypoToolConfig_fastreduction.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetHypoToolHelperNoGrouper.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetHypoToolHelperNoGrouper.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetPassThroughFilterConfig.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetPassThroughFilterConfig.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetRangeFilterConfig.cxx
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/TrigJetRangeFilterConfig.h
- Trigger/TrigHypothesis/TrigHLTJetHypo/src/components/TrigHLTJetHypo_entries.cxx
added JetEtmiss Trigger review-pending-level-1 labels
CI Result FAILURE (hash d3f0ca6c)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon 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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30230]added review-user-action-required label and removed review-pending-level-1 label
removed review-user-action-required label
mentioned in merge request !42266 (merged)
closing - work has moved to !42266 (merged)