-
Notifications
You must be signed in to change notification settings - Fork 328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Undo JEC for subjets in FatJet #281
base: master
Are you sure you want to change the base?
Conversation
FYI JMAR convenors @camclean @alefisico |
can you document the change with a plot comparing the SDmass after correction and the raw ? |
@mariadalfonso will do |
The attached plot compares the fatjet soft-drop mass distribution for the following cases:
As shown in the plot, the Raw (Before Fix) curve overlaps exactly with Nano because currently we do not undo the jet energy correction on the subjets. With this PR, the raw soft-drop mass will be properly calculated. FatJets with pT > 200 GeV, |eta| < 2.4 and have 2 subjets are considered. The input file used for this plot is |
Can you also compare |
groomedP4 = subJets[jet.subJetIdx1].p4() + subJets[ | ||
jet.subJetIdx2].p4() # check subjet jecs | ||
groomedP4 = (subJets[jet.subJetIdx1].p4() * (1. - subJets[jet.subJetIdx1].rawFactor)) \ | ||
+ (subJets[jet.subJetIdx2].p4() * (1. - subJets[jet.subJetIdx2].rawFactor)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you check that the eta and phi of the jet are not affected? The raw factor should only be applied to the pt and mass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@camclean Are you referring to eta and phi of the AK8 parent jet or each of the subjets?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The one you get from groomedP4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The attached slide shows pT, eta and phi distributions of subjet 1 (top) and subjet 2 (bottom). Black solid histogram is with JEC applied (by default in NanoAOD) and blue dashed histogram is without the JEC applied (as implemented by the fixed in this PR). The implementation does not affect the subjets' eta and phi variables.
There are also these lines after your fix that affect the "raw"softdrop mass. |
Attached is a plot comparing the Note: FatJets with pT > 200 GeV, |eta| < 2.4 and have 2 subjets are considered. The input file used for this plot is |
So now we have:
The question is whether or not we want to keep msoftdrop_nom with JES applied for consistency. We've had analyses in the past have issues with changes because there is no version control for nanoAOD-Tools. What do you think @mariadalfonso @alefisico? |
From the plots that @nurfikri89 showed, I think we should only keep the |
Agreed |
@alefisico @camclean @nurfikri89 msoftdrop_nom (After fix): JER, JMR, JMS, and PUPPI softdrop mass correction applied; no JES applied can you document
|
Just linking that this issue was raised a year and a half ago by #234 |
Shouldn't this PR to be merged to master? |
@mariadalfonso My apologies for dropping this PR. Picking this back up again so it can be merged. The JMR and JMS values used for the plot in #281 (comment) are 1.0 which are the default values for UL2017 as defined here for JMR and here for JMS. |
for the SF, it's safest to take the value from the json files from JMAR-EOY. |
This PR is to address issue #280. The subjets' p4 should now have the raw (i.e no JEC applied on them) p4 and the raw softdrop mass can be correctly calculated.