filters for charged hadrons and muons that have bad track quality cau…

[clelange]

…sing very high-pT jets and MET tails
If all PF bugs are fixed in 76X, this should not filter any events.

[cmsbuild]

A new Pull Request was created by @clelange for CMSSW_7_6_X.

It involves the following packages:

RecoMET/METFilters

@cmsbuild, @vadler, @davidlange6, @monttj 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.

Following commands in first line of a comment are recognized

• +1|approve[d]|sign[ed]: L1/L2's to approve it
• -1|reject[ed]: L1/L2's to reject it
• assign <category>[,<category>[,...]]: L1/L2's to request signatures from other categories
• unassign <category>[,<category>[,...]]: L1/L2's to remove signatures from other categories
• hold: L1/all L2's/release manager to mark it as on hold
• unhold: L1/user who put this PR on hold
• merge: L1/release managers to merge this request
• [@cmsbuild,] please test: L1/L2 and selected users to start jenkins tests
• [@cmsbuild,] please test with cms-sw/cmsdist#<PR>: L1/L2 and selected users to start jenkins tests using externals from cmsdist

[gpetruc]

Hi,

Shouldn't the paths be also added to PhysicsTools/PatAlgos/python/slimming/metFilterPaths_cff.py ?

Giovanni

[cmsbuild]

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

[schoef]

@gpetruc Definitely. Should come today.

[clelange]

Hi Giovanni, I added it in clelange@102ed50
Clemens

[cvuosalo]

@cmsbuild please test

[cmsbuild]

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

[slava77]

should be thread-aware
better be edm::global::EDFilter
but may be faster to say edm::stream

[cmsbuild]

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

[clelange]

Hi Slava,

I think I should have implemented your comments on the code.

Clemens

[cmsbuild]

+1
Tested at: 102ed50
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12515/10080/summary.html

[slava77]

@cmsbuild please test

[cmsbuild]

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

[cmsbuild]

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

[cmsbuild]

+1
Tested at: d63d2a9
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12515/10093/summary.html

[cmsbuild]

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

[slava77]

@clelange please make a PR for 80X as well
If this gets merged first and we see that auto-forward-port to 80X went well that 80X PR could be closed.

[slava77]

+1

for #12515 d63d2a9

• new filters added. The code looks somewhat reasonable. An alternative to selections made in the created filters is still desirable, since essentially all of it can be applied on either AOD or miniAOD.
• jenkins tests pass; the comparisons with the baseline show no differences.

[slava77]

@schoef @mariadalfonso does it actually make sense to add the two new filters to metFilters sequence, the "combined MET filters" result?