New HCalTower cleaner including rechits

[safarzad]

New module includes rechit filter to be applied in 2016 menus

[cmsbuild]

A new Pull Request was created by @safarzad for CMSSW_8_0_X.

New HCalTower cleaner including rechits

It involves the following packages:

HLTrigger/JetMET

@Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald 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' or '@cmsbuild, please test' 'please test with cms-sw/cmsdist#PR' 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.

[Martin-Grunewald]

Hi,

You also need to declare it as a plugin using the Fw macros.

[safarzad]

I do not know what is Fw macros

[safarzad]

it is only a new module that if it is in cmssw we will be able to go further and create a JIRA ticket for HLT cfg to use this module in 2016.

[Martin-Grunewald]

You will not be able to use it with cmsRun if it is not declared as a plugin:
Add it to https://github.com/cms-sw/cmssw/blob/CMSSW_8_0_X/HLTrigger/JetMET/plugins/SealModule.cc - see there how this is done for the other modules.
(and read up on the FwEDM twikis on how to declare new plugins).

[Martin-Grunewald]

please test

[cmsbuild]

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

[cmsbuild]

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

[cmsbuild]

+1
Tested at: f6687ae
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12334/9580/summary.html

[Martin-Grunewald]

+1

[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. @davidlange6, @Degano, @smuzaffar

[cmsbuild]

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

[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. @davidlange6, @Degano, @smuzaffar