METfilters: fix forth bad EE supercluster

[mariadalfonso]

This PR refine one MET filter adding one bad SC to the list. See finding / validation in
https://hypernews.cern.ch/HyperNews/CMS/get/ecal-performance/839/2/1.html
We aim to have in the 76 re-reco of the data
@schoef

[cmsbuild]

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

METfilters: fix forth bad EE supercluster

It involves the following packages:

RecoMET/METFilters

@cmsbuild, @vadler, @monttj can you please review it and eventually sign? Thanks.
@TaiSakuma, @ahinzmann, @mmarionncern, @jdolen, @nhanvtran, @schoef this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[slava77]

@cmsbuild please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: f0f0cf8
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12034/9116/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
69b9e4c
20f305b
76acbb8
b5c0254
8b91937
9c46360
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12034/9116/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12034/9116/git-merge-result

[cmsbuild]

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

[slava77]

Maria,
if I understand correctly, the MET filter decisions are not monitored in DQM/Validation.
If that's correct, why not?

[mariadalfonso]

@slava77 @webermat
Hi Slava. For now in the DQM we have two histograms of pfMET (with the filters applied and not), matthias knows the details. This should be enough for the data quality monitors of the runs. For the release integration do you desire to have histograms to monitor the pass fails for each of the standard filters (HBHE, HBHEIso, EESC, CSCTigh2015)?

[slava77]

I think that ta a histogram with filter pass/fail count would be useful
(it could be just one histogram for all filters with pass counts filled; could be separate histograms with 2-bin pass/fail).
Memory-wise, one histogram with a bin per filter would be better.

[mariadalfonso]

@slava77 @webermat
ok, the DQM modification you propose can be added in the pending DQM update
#11539
and in the meanwhile I propose to go with the merging of this PR

[slava77]

I checked jenkins outputs and observe no difference.
This just says the problematic cell shows up rarely enough.
This PR has no RECO signature requirement (maybe it eventually should, since this code is a part of RECO+miniAOD workflow).
It looks fine for me.

@davidlange6