Pixel Phase1 xmls for 72X

[boudoul]

First round of XMLs files for 72X : starting with pixel phase1 synchronized with the latest developments in 62X_SLHC (@mark-grimes & @venturia : FYI, starting to update 72X with the upgrade geometries)
@atricomi : any xml updates should be kept synchronized with 7x from now

[cmsbuild]

A new Pull Request was created by @boudoul (boudoul) for CMSSW_7_2_X.

Pixel Phase1 xmls for 72X

It involves the following packages:

Geometry/TrackerCommonData

@civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @ktf can you please review it and eventually sign? Thanks.
@ghellwig, @VinInn 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.

[cmsbuild]

+1
Tested at: ffd37e0
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5133/770/summary.html

[ianna]

It's better to remove it.

[boudoul]

Done in my new commit, thanks
Le 2 sept. 2014 à 09:08, ianna a écrit :

In Geometry/TrackerCommonData/data/PhaseI/pixbar.xml:

@@ -1059,7 +1023,9 @@

+
It's better to remove it.

—
Reply to this email directly or view it on GitHub.

[ianna]

@boudoul - which scenario does it affect? Should a DB payload be updated as well?

[cmsbuild]

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

[boudoul]

it's for any scenario >=2017.

There no upgrade GT for the upgrade in 72X so no DB payload to update at this point.

Thanks
Gaelle

Le 2 sept. 2014 à 09:10, ianna a écrit :

@boudoul - which scenario does it affect? Should a DB payload be updated as well?

—
Reply to this email directly or view it on GitHub.

[cmsbuild]

Pull request #5133 was updated. @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @ktf can you please check and sign again.

[ianna]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests.

[cmsbuild]

+1
Tested at: f35db24
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5133/775/summary.html

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes (tests are also fine).