Pcc 90 x

[nfilipov]

This PR affects only the Pixel Cluster Counting method. In this PR we:

• add a switch to be respectful of the Phase2 tracker geometry (number of pixel barrel layers)
• remove some Run 1 unused stuff (commented out already)
• replace the TrackerGeometry::DetContainer loop, with an edmNew::DetSet loop that has the same effect.

[cmsbuild]

A new Pull Request was created by @nfilipov (Nicolas Filipovic) for CMSSW_9_0_X.

It involves the following packages:

RecoLuminosity/LumiProducer

@cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@davidlange6, @smuzaffar you are the release manager for this.

cms-bot commands are listed here #13028

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/18017/console Started: 2017/02/28 18:06
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/18018/console Started: 2017/02/28 19:42

[cmsbuild]

+1
Tested at: 3cb09eb
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-17647/18018/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

[slava77]

I'm curious why is this only for phase2?
Doesn't phase-1 have the same feature?

[slava77]

+1

for #17647 3cb09eb

• test/analysis code, not used in standard/central workflows
• jenkins tests pass: compiles

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_0_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[davidlange6]

+1