-
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
Updating JEC in data/mc and fixes for HI #12349
Conversation
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @mmusich (Marco Musich) for CMSSW_7_6_X. Updating JEC in data/mc and fixes for HI It involves the following packages: Configuration/AlCa @cmsbuild, @mmusich, @diguida, @franzoni, @cerminar can you please review it and eventually sign? Thanks. |
79e100b
to
703cffe
Compare
@cmsbuild please test |
The tests are being triggered in jenkins. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_6_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
a bit surprised that nothing seems to have changed in the comparisons of this PR.. Approving but might be worth a double check as I would have expected the DQM to consume the new JEC (naively) |
Updating JEC in data/mc and fixes for HI
@davidlange6 as far as I could understand the JETMET POG provided new payloads with updates concerning only the uncertainties (used in the miniAOD and possibly nowhere else). Please @blinkseb @schoef confirm. |
No changes expected in MC since MC JEC are identical. Uncertainties are presumably not tested. |
Summary of changes in GT
RunII simulation
RunI data
RunII data