optimized CPU timing of ECAL Pi0 stream

[cippy]

Optimized CPU timing of ECAL Pi0 stream, following request in [0]. Changes affect HLTRegionalEcalResonanceFilter::calcPairCluster(...) method.
Tested on ZeroBias2016 dataset: CPU time reduced by a factor

• ~ 30% in Barrel
• ~ 50% in Endcap

[0] https://its.cern.ch/jira/browse/CMSHLT-741

@emanueledimarco @shervin86 @crovelli

[cmsbuild]

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

It involves the following packages:

HLTrigger/special

@Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard, @davidlange6 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.

cms-bot commands are list here #13028

[Martin-Grunewald]

81X PR needed as well!

[Martin-Grunewald]

please test

[cmsbuild]

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

[fwyzard]

could you use math::XYZVector instead of TVector3 ?

[cippy]

@fwyzard It could be done, but it need to be validated. Some time needed. Could you first integrate this in the meantime?

@Martin-Grunewald pull request for 8_1_X coming

[fwyzard]

@fwyzard It could be done, but it need to be validated. Some time needed. Could you first integrate this in the meantime?

Please do. I don't think we need to push this online within the next couple of days anyway, right ?

[emanueledimarco]

@fwyzard Right. If it is not so urgent, then @cippy can look at it.

[cmsbuild]

+1
Tested at: 6306e7f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14358/12796/summary.html

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14358/12796/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. @slava77, @davidlange6, @Degano, @smuzaffar

[cmsbuild]

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

[cmsbuild]

Pull request #14358 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @fwyzard, @davidlange6 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/12859/console

[cmsbuild]

+1
Tested at: 3341cd5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-14358/12859/summary.html

[cmsbuild]

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

[fwyzard]

@cippy thanks for the latest changes.

.A

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

[davidlange6]

+1