Switch gen jet collection to be those with no neutrinos. #423
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
|
Tagging @mandrenguyen and @jusaviin so they can have a look. |
|
Let me point out that I also just submitted a PR to CMSSW to exclude neutrinos from genJets in miniAOD: cms-sw#47210 So for next year's data taking when we don't recluster the reco jets we will be able to also not recluster the genJets. |
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.
PR description:
This PR switches the gen jet collection to be that which does not include neutrinos. This is to correct a mismatch between ref and gen jets, leading to incorrect determination of
genmatchindex. This will be corrected centrally by cms-sw#47210, but this serves as a temporary fix for the forest.PR validation:
This PR was validated by performing a small test, which removes the observed precision issues as seen in the screenshot below.
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
This PR does not need a backport.