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
updated interpolation tables #2980
Conversation
A new Pull Request was created by @igv4321 for CMSSW_7_1_X. updated interpolation tables It involves the following packages: JetMETCorrections/InterpolationTables @nclopezo, @vadler, @cmsbuild, @Degano, @monttj can you please review it and eventually sign? Thanks. |
@igv4321 : Have these changes documented/presented somewhere? Where should effedcts show up? |
The changes are trivial -- mostly maintenance, so there is no detailed documentation. They affect my packages related to FFTJet jet corrections, mainly JetMETCorrections/FFTJetObjects and JetMETCorrections/FFTJetModules. As far as I can see, everything works just fine. |
I see. So, you expect output to be the same as before, and it indeed is? |
Yes, the code works the same as before. Interpolation tables do not generate output directly -- they are just used to store lookup tables of correction factors. |
+1 |
AT -- Updated JetMETCorrections/InterpolationTables
A few updates to the interpolation tables, some cleanup, explicit constructors where needed, etc