[92X] Fix pixel barycentre tool

[ghellwig]

• fix use case of alignment producer that is needed for the bary-centre tool
• make the tool geometry independent
• streamline tool script
• added a unit test for the tool

[ghellwig]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19853/console Started: 2017/05/15 22:47

[cmsbuild]

A new Pull Request was created by @ghellwig (Gregor Mittag) for master.

It involves the following packages:

Alignment/CommonAlignmentProducer
Alignment/MillePedeAlignmentAlgorithm
Alignment/TrackerAlignment

@ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @pakhotin, @tocheng, @tlampen, @mschrode, @mmusich this is something you requested to watch as well.
@davidlange6 you are the release manager for this.

cms-bot commands are listed here

[ghellwig]

type bugfix

[cmsbuild]

+1
Tested at: 9b750e1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18754/19853/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 3361 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1829394
• DQMHistoTests: Total failures: 47225
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1781989
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[ghellwig]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/19867/console Started: 2017/05/16 18:45

[cmsbuild]

Pull request #18754 was updated. @ghellwig, @arunhep, @cerminar, @cmsbuild, @franzoni, @mmusich, @davidlange6 can you please check and sign again.

[franzoni]

no unexpected changes observed in the tests available for 10024.0_TTbar , 136.731_RunSinglePh2016B and 10042.0_ZMM
(the commit added after won't change any of the official sequences)

[franzoni]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @smuzaffar

[cmsbuild]

+1
Tested at: 84ebb4a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18754/19867/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 3324 differences found in the comparisons
• DQMHistoTests: Total files compared: 24
• DQMHistoTests: Total histograms compared: 1833892
• DQMHistoTests: Total failures: 33213
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 1800499
• DQMHistoTests: Total skipped: 180
• DQMHistoTests: Total Missing objects: 0
• Checked 98 log files, 14 edm output root files, 24 DQM output files

[davidlange6]

+1