APE tool implemented

[cschomak]

• APE tool implemented into the release
• TrackerTreeGenerator (necessary for the APE tool, but might also used for other purposes) included in Alignment/TrackerAlignment
• Fixes for ApeSettingAlgorithm in CommonAlignmentAlgorithm and AlignmentProducer in CommonAlignmentProducer

[cmsbuild]

A new Pull Request was created by @cschomak for CMSSW_7_6_X.

APE tool implemented

It involves the following packages:

Alignment/APEEstimation
Alignment/CommonAlignmentAlgorithm
Alignment/CommonAlignmentProducer
Alignment/TrackerAlignment

The following packages do not have a category, yet:

Alignment/APEEstimation

@cmsbuild, @mmusich, @diguida, @cerminar can you please review it and eventually sign? Thanks.
@ghellwig, @pakhotin, @frmeier, @tlampen, @mschrode, @mmusich 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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[mmusich]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.

[mmusich]

@cschomak I see that there are several examples of cff in Alignment/APEEstimation/python/samples/. It's good to have those for documentation purposes (maybe linked in a Twiki), but I don't think they all deserve to be included in release. Can you trim down the list to the bare minimum?

[cmsbuild]

Pull request #10601 was updated. @cmsbuild, @mmusich, @diguida, @cerminar can you please check and sign again.

[mmusich]

@cschomak thank you!

[cmsbuild]

+1
Tested at: c618390
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10601/193776/summary.html

[cmsbuild]

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

[davidlange6]

@cschomak - the producers/analyzers in this PR need to have consumes statements for all of the products they consume (and ideally use getbytoken instead of getbylabel)

[cschomak]

@davidlange6 - I am not sure what you mean by by consumes statements. Could you give an example?

[mmusich]

@cschomak this is not updated, please move to a more recent GT in condDB v2

[mmusich]

this file Alignment/APEEstimation/test/cfgTemplate/produceOfflineValidationTex.py duplicates Alignment/OfflineValidation/scripts/produceOfflineValidationTex.py? Can you please remove duplications?

[cmsbuild]

Pull request #10601 was updated. @cmsbuild, @mmusich, @diguida, @cerminar can you please check and sign again.

[cschomak]

@mmusich I was probably not strict enough when cleaning up thinks I did not need yet. Some are outdated and I think Ajay included some to test the tool, but they are not required.
I removed these files now and changed the config as you suggested

[mmusich]

please test

[mmusich]

+1
provided jenkins agrees

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 8fe169a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10601/7153/summary.html

[cmsbuild]

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

[davidlange6]

+1