-
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
PFMet and PuppiMET Branches: Backport to CMSSW_141X #44383
Conversation
Adding the rivetMETTable Co-authored-by: Markus Seidel <[email protected]>
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44383/39444
|
A new Pull Request was created by @wajidalikhan for master. It involves the following packages:
@hqucms, @tjavaid, @vlimant, @nothingface0, @syuvivida, @jfernan2, @cmsbuild, @rvenditti, @antoniovagnerini, @mandrenguyen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@wajidalikhan I think this is a backport of #44335, not #44038. Please update the PR title and description accordingly. |
Can you please update the PR description though? Please keep the list of PRs explicit rather than hiding the PR#s in hyper-lnks. |
The milestone seems to be set to 14_1_X rather than 14_0_X |
please overhaul this backport (cherry pick the proper commit and make it against 14_0_X) asap @wajidalikhan |
The 14_0_X backport is done in #44495. |
please close |
PR description:
This is a back-port to CMSSW_141X for the PR PFMet and PuppiMET Branches #44038 which incorporates
PFMet and PuppiMET do not have the same variables stored in NanoAOD, consistent variables to be stored in both the PFMET and PuppiMET at the NanoAOD for next data taking
Remove / Rename some of the other relevant branches
Had to do the cherry picking of the https://github.com/cms-sw/cmssw/pull/44038/commits because unknown issues