Update to MVA MET: Fullfills "consumes" call requirement, Bugfix for the calculation of the covariance matrix, updated path to weight files. Same changes as in PR for CMSSW76X

[rfriese]

Same commit as this one:
#13086

[cmsbuild]

A new Pull Request was created by @rfriese (Raphael Friese) for CMSSW_8_0_X.

It involves the following packages:

RecoMET/METPUSubtraction

@cmsbuild, @cvuosalo, @davidlange6, @slava77 can you please review it and eventually sign? Thanks.
@TaiSakuma, @ahinzmann, @mmarionncern, @jdolen, @nhanvtran, @schoef, @mariadalfonso this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[slava77]

@rfriese
please edit the PR description block and give more details on the changes (plots or a link to a presentation in a POG meeting)

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10777/console

[cmsbuild]

+1
Tested at: a1f1b97
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13088/10777/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
1aabae2
516f419
e6efb5a
8553987
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13088/10777/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13088/10777/git-merge-result

[cmsbuild]

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

[slava77]

@rfriese
please update the PR description here as well (descriptive text and a link to the 76X PR would suffice)

[cmsbuild]

Pull request #13088 was updated. @cmsbuild, @cvuosalo, @davidlange6, @slava77 can you please check and sign again.

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10812/console

[cmsbuild]

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

[cmsbuild]

Pull request #13088 was updated. @cmsbuild, @cvuosalo, @davidlange6, @slava77 can you please check and sign again.

[cvuosalo]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/10894/console

[cmsbuild]

+1
Tested at: 04e5ffc
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13088/10894/summary.html

[cmsbuild]

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

[cvuosalo]

+1

For #13088 04e5ffc

Bug fix for MVA MET covariance matrix calculation. #13086 is the 76X version of this PR, and it has already been approved.

The code changes are satisfactory, and Jenkins tests against baseline CMSSW_8_0_X_2016-02-01-1100 show no significant differences.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @Degano, @smuzaffar

[cvuosalo]

Reference note: Testing shows that this PR does not change the size of Mini-AOD event content.