Add DeepMET into NanoAOD

[yongbinfeng]

PR description:

The PR is to add DeepMETs into NanoAOD, following the DeepMET integration into MiniAOD here: #29764.

(And more information about DeepMET could be found from presentations in the JMET group:
https://indico.cern.ch/event/912067/contributions/3835851/ (most recent update)
https://indico.cern.ch/event/883809/contributions/3733818/ (CMS week JetMET meeting)
https://indico.cern.ch/event/854654/contributions/3594579/ (first presentation in MET meeting)
)

The PR will add 4 more variables in NanoAOD (pt and phi of DeepMETResolutionTune and DeepMETResponseTune).

If running with the past Run2 eras where DeepMETs are not saved in MiniAOD, the DeepMET producer is called and DeepMETs will be computed on the fly and saved into Nano. For the future we expect them to be already computed and saved in MiniAOD, so we use the extractions instead, in a similar way as patCaloMet.

The PR was initially made towards cms-nanoAOD. Then Andreas suggests making one here since cms-nanoAOD is out of sync.

@steggema @intrepid42

@ahinzmann @lathomas

PR validation:

test on a small sample and it works as expected. Also passed unit tests and the runTheMatrix tests.

if this PR is a backport please specify the original PR and why you need to backport that PR:

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30291/16242

• This PR adds an extra 52KB to repository

• There are other open Pull requests which might conflict with changes you have proposed:

  • File PhysicsTools/PatAlgos/plugins/RecoMETExtractor.cc modified in PR(s): [RFC] Test clang-tidy --checks modernize-use-auto #29936

[cmsbuild]

A new Pull Request was created by @yongbinfeng (Yongbin Feng) for master.

It involves the following packages:

PhysicsTools/NanoAOD
PhysicsTools/PatAlgos
PhysicsTools/PatUtils

@perrotta, @cmsbuild, @fgolf, @slava77, @santocch, @peruzzim can you please review it and eventually sign? Thanks.
@rappoccio, @gouskos, @hatakeyamak, @emilbols, @peruzzim, @seemasharmafnal, @mmarionncern, @ahinzmann, @smoortga, @jdolen, @ferencek, @jdamgov, @nhanvtran, @gkasieczka, @schoef, @andrzejnovak, @clelange, @riga, @JyothsnaKomaragiri, @mverzett, @gpetruc, @mariadalfonso this is something you requested to watch as well.
@silviodonato, @dpiparo you are the release manager for this.

cms-bot commands are listed here

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

• CMSSW_11_2_X_2020-06-18-1100/slc7_amd64_gcc820: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/7182/console Started: 2020/06/18 18:18

[cmsbuild]

+1
Tested at: 9ba59d6
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e3f58e/7182/summary.html
CMSSW: CMSSW_11_2_X_2020-06-18-1100
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e3f58e/7182/summary.html

Comparison Summary:

• No significant changes to the logs found
• ROOTFileChecks: Some differences in event products or their sizes found
• Reco comparison results: 8 differences found in the comparisons
• DQMHistoTests: Total files compared: 36
• DQMHistoTests: Total histograms compared: 2778811
• DQMHistoTests: Total failures: 7
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2778754
• DQMHistoTests: Total skipped: 50
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.51 KiB( 35 files compared)
• DQMHistoSizes: changed ( 1325.7 ): 0.510 KiB Physics/NanoAODDQM
• Checked 152 log files, 16 edm output root files, 36 DQM output files

[santocch]

+1

[slava77]

@yongbinfeng
was this presented in a JME or Cross-POG meeting?

[cmsbuild]

The tests are being triggered in jenkins.

• CMSSW_11_2_X_2020-07-29-2300/slc7_amd64_gcc820: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/8426/console Started: 2020/07/30 15:20

[cmsbuild]

+1
Tested at: bd5f160
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e3f58e/8426/summary.html
CMSSW: CMSSW_11_2_X_2020-07-29-2300
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e3f58e/8426/summary.html

Comparison Summary:

• No significant changes to the logs found
• ROOTFileChecks: Some differences in event products or their sizes found
• Reco comparison results: 2 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2525444
• DQMHistoTests: Total failures: 5
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 2525391
• DQMHistoTests: Total skipped: 47
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.514 KiB( 33 files compared)
• DQMHistoSizes: changed ( 10224.0 ): 0.004 KiB MessageLogger/Warnings
• DQMHistoSizes: changed ( 1325.7 ): 0.510 KiB Physics/NanoAODDQM
• Checked 144 log files, 17 edm output root files, 34 DQM output files

[mariadalfonso]

+1

[mariadalfonso]

unhold

[slava77]

+1

for #30291 bd5f160

• updates since the last signoff for 00b0809 were in nanoAOD PhysicsTools/NanoAOD/python/met_cff.py, not affecting reco category files/execution

[silviodonato]

merge

[santocch]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will be automatically merged.