Exclusive jet selections #47
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
First implementation of exclusive jet finder, for njets Add a record of initial jets/clusters given to the ClusterSequence. Add an Enum for the jet algorithm (to be allowed as a CLI option and also to record this value internally in the ClusterSequence).
Used to store the algorithm in the ClusterSequence Can also be used to build a simple CLI to select the algorithm
Call this RecoStrategy instead if JetRecoStrategy (This is different from JetAlgorithm, where it's how do I define jets, vs. RecoStrategy, how to I build the reconstructed jets - seems to make sense to me...)
This returns the number of jets that would pass a cut where the reconstruction stopped at a certain dcut value
This makes comparisons a lot easier!
Had been multiplying by R^2, instead of dividing!
Inclusive kT with dijmax=20
Better structuring for exclusive selection tests using a struct for all parameters. Tests for njets and dijcut for both Cambridge/Aachen and Inclusive-kT.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add exclusive jet selection methods. These take a
ClusterSequenceobject and return the exclusive jets based on either annjetscut (that is, cluster up tonjetsand stop) or on adijmaxcut (that is, cluster until the metric distance exceeds that cut value).There is an auxiliary method which returns the number of jets for a particular
dijcut value.Four tests are added, which test
njetsanddijmaxfor the Cambridge/Aachen and Inclusive-kT algorithms (N.B. exclusive selections are not useful for anti-kT).Also changed is to rename
JetRecoStrategytoRecoStrategy.Closes #41