Closed jkarancs closed 8 years ago
@decosa @dmajumder @rappoccio Can any of you, please review the proposed changes? Thank you in advance!
Ok, I changed it. I was actually thinking about the exact same name at one point. :)
Sorry to poke at an old topic, but it seems like you only apply 'scaling' as defined by https://twiki.cern.ch/twiki/bin/view/CMS/JetResolution#Smearing_procedures, not the hybrid scaling+smearing. Am I just not seeing where this is applied?
Hello Susan,
Yes, you are right, I left that part of the code (including the avriable name) intact. It seems what we previously called "smearing" was actually scaling, I merely replaced the variables where we take the scale factor from. In principle, one can do the recipe on the twiki (which apparently was updated recently) during the analysis, because we save gen jets, sigma (resolution) and the scale factors in the ntuples, so if one needs to use this procedure, he/she can do it. A renaming of the variable, or implementing the recommended recipe could be useful if the recipe does not change further.
Changes include:
UPDATE (2nd commit):