adds HLTPixelActivityHFSumEnergyFilter module (superseeds #12483)

[mverwe]

This PR superseeds #12483

Adds HLTPixelActivityHFSumEnergyFilter module. This filter checks correlation between number of pixel clusters and HF energy sum. The idea is to have a filter firing when the expected correlation is broken.

Remark 1 : This module is NOT for the pp reference run.
Remark 2 : This module is for PbPb run.

@kkrajczar
@icali
@ttrk

[cmsbuild]

A new Pull Request was created by @mverwe (Marta Verweij) for CMSSW_7_5_X.

It involves the following packages:

HLTrigger/special

@Martin-Grunewald, @perrotta, @cmsbuild, @davidlange6, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald 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

[slava77]

@cmsbuild please test

[slava77]

@mverwe please use "Edit" button at the top and change the title of the PR to something meaningful and self-descriptive

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 2c0ec9c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12487/9835/summary.html

[cmsbuild]

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

[Martin-Grunewald]

This must be a normal (stream or global) EDFilter, not an HLTFilter, as it does not (and can not!) save any TriggerObjects. Please make the change.

[Martin-Grunewald]

-1

[Martin-Grunewald]

please test

[cmsbuild]

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

[mverwe]

We've addressed all comments. Let us know in case of further remarks.

[cmsbuild]

Pull request #12487 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @davidlange6, @fwyzard can you please check and sign again.

[Martin-Grunewald]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 4c5d28b
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12487/9853/summary.html

[cmsbuild]

+1
Tested at: 7e399e2
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12487/9859/summary.html

[cmsbuild]

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

[Martin-Grunewald]

+1

[cmsbuild]

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

[cmsbuild]

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

[slava77]

merge

all new code

[Martin-Grunewald]

BTW, is there a PR for 76X as well? Please make one!

[mverwe]

@Martin-Grunewald The PR for 76X will be done asap