integrate the fast multifit for HLT in 740pre5, yet not using the DB inp...

[emanueledimarco]

...uts.

The default behavior is the same as before, so no change is expected. The fast version of the local reconstruction will be triggered by changing the parameters doPrefitEB and doPrefitEE via the HLT menu.

[emanueledimarco]

@argiro @bendavid @fwyzard @gennai @matteosan1 @cerminar This is an intermediate PR in 7.4.X branch integrating the fast multifit version for the HLT, in order to have it in an earlier IB of 7.4.X than the last open pre7.

It still uses the inputs from python, not being dependent on the new payloads containing ECAL templates, that will be pushed when the GT will be ready.

[cmsbuild]

A new Pull Request was created by @emanueledimarco (Emanuele Di Marco) for CMSSW_7_4_X.

integrate the fast multifit for HLT in 740pre5, yet not using the DB inp...

It involves the following packages:

RecoLocalCalo/EcalRecAlgos
RecoLocalCalo/EcalRecProducers

@cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks.
@argiro 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.

[fwyzard]

@cvuosalo, @slava77, apologies, but I have to ask you if you could test and sign this as quickly as possible: we need to have it in an IB to migrate the HLT 25ns menu to the fast multifit method .

[fwyzard]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 10a9446
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7441/2214/summary.html

[cmsbuild]

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

[slava77]

I don't see any differences in DQM or direct-fwlite comparisons in the short tests done by jenkins.
Is this expected?
If so, please update the description of the PR to mention that the added feature is not enabled and no changes are expected.

[emanueledimarco]

@slava77 Yes, it is expected. The PR contains the code and the necessary switches to enable the "fast version" for the HLT: the default parameters correspond to the pre-PR configuration and will be changed directly via the HLT menu. I am updating the description of the PR.

[slava77]

+1

for #7441 10a9446
jenkins confirms no differences as expected
the code looks OK, the technical performance evaluation will happen when it is enabled

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged.