Skip to content
This repository has been archived by the owner on Sep 5, 2020. It is now read-only.

Commit

Permalink
Fix a bug in stochastic JER smearing
Browse files Browse the repository at this point in the history
Resolution in pt is relative, so should not divide by pt.
  • Loading branch information
andrey-popov committed Aug 3, 2016
1 parent da86372 commit 7967a1d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/extensions/src/JetCorrectorService.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ double JetCorrectorService::Eval(Jet const &jet, double rho, SystType syst /*= S
jerProvider->getResolution({{JME::Binning::JetPt, jecCorrPt},
{JME::Binning::JetEta, jet.Eta()}, {JME::Binning::Rho, rho}});
double const jerFactor = 1. + rGen->Gaus(0., ptResolution) *
std::sqrt(std::max(std::pow(jerSF, 2) - 1., 0.)) / jecCorrPt;
std::sqrt(std::max(std::pow(jerSF, 2) - 1., 0.));

corrFactor *= jerFactor;
}
Expand Down

0 comments on commit 7967a1d

Please sign in to comment.