-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Disabling the reclustering in the MET correction and uncertainty tool #11489
Disabling the reclustering in the MET correction and uncertainty tool #11489
Conversation
A new Pull Request was created by @mmarionncern for CMSSW_7_4_X. Disabling the reclustering in the MET correction and uncertainty tool It involves the following packages: JetMETCorrections/Type1MET @cmsbuild, @vadler, @monttj can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
Hi @mmarionncern, This PR affects only the re-computation of type1 MET from miniAOD, not what's run in prooduction, ok? Giovanni |
@gpetruc Yes, the type1MET correction is only computed at the miniAOD level so far. |
src = jetCollection, | ||
levels = ['L1FastJet', | ||
'L2Relative', | ||
'L3Absolute'], |
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.
Naive question: shouldn't there be also L2L3Residual?
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.
There is a specific case for data L1252 of the new version
Comparison is ready The workflows 140.53 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons |
Disabling the reclustering in the MET correction and uncertainty tool
Following the discussion of https://indico.cern.ch/event/447792/?filterActive=1&showDate=all&showSession=4
this PR removes the jet clustering and enable the repro74X option in the shortcut functions used for MET reprocessing on top of miniAOD.
New features :
Changes expected :
Has to be done on top of #11482