Removing SiStripDetIds dependencies on SiStripTKNumbers. Replacing Si…

[vidalm]

cc @alesaggio @OlivierBondu @pieterdavid

Removing SiStripDetIds dependencies replaced by DetId objects.

To the experts (@boudoul) please advice if some test is needed to double check that everything works within DPGAnalysis.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22251/3412

[cmsbuild]

A new Pull Request was created by @vidalm (Miguel Vidal) for master.

It involves the following packages:

DPGAnalysis/SiStripTools

@cmsbuild, @monttj can you please review it and eventually sign? Thanks.
@ebrondol, @threus, @mmusich, @venturia this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[pieterdavid]

This is tracked at #19398

[fabiocos]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/26142/console Started: 2018/02/17 12:54

[cmsbuild]

+1
Tested at: 8104980
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22251/26142/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 27
• DQMHistoTests: Total histograms compared: 2465360
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2465190
• DQMHistoTests: Total skipped: 169
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 1.15999999983 KiB( 22 files compared)
• Checked 111 log files, 9 edm output root files, 27 DQM output files

[fabiocos]

+1

[fabiocos]

@alesaggio @OlivierBondu @pieterdavid @boudoul @vidalm technically the code looks ok to me, please comment in case before I merge

[fabiocos]

merge