Phase2-hgx101 Try to introduce the new geometry of HGCal - only done up to EE so far

[bsunanda]

Introduce the creation of wafers with the new layout and the layers for the EE part

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22398/3625

[cmsbuild]

A new Pull Request was created by @bsunanda for master.

It involves the following packages:

Geometry/HGCalCommonData

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

[bsunanda]

@cmsbuild Please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/26392/console Started: 2018/03/01 01:22

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-22398/3626

[cmsbuild]

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

[davidlange6]

please test

[cmsbuild]

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

[cmsbuild]

+1
Tested at: 110051a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-22398/26483/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 29
• DQMHistoTests: Total histograms compared: 2479021
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2478844
• DQMHistoTests: Total skipped: 176
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.960000000072 KiB( 22 files compared)
• Checked 118 log files, 9 edm output root files, 29 DQM output files

[bsunanda]

@ianna @kpedro88 Could you approve this PR?

[kpedro88]

+1

[bsunanda]

@ianna @civanch Please approve this. It is the first step for new geometry of HGCal. I have to do this in steps because of complexity of the new design

[ianna]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[fabiocos]

@bsunanda @kpedro88 is this code practically run in one of the tested upgrade scenarios, or are we just testing that it doesn't break DDHGcalEEAlgo (as I guess, since the v9 version is not present up to now, and I do not see associated changes in the test configurations)? As a starting point development for Phase2 it could be ok, but for my understanding I would like a confirmation

[kpedro88]

@fabiocos the latter - just an intermediate PR to build up for further development

[bsunanda]

@fabiocos - we cannot have a full upgrade scenario with this for a while - first to complete the G4 geometry part, then redefine DetID's, then interface with SIM step, then the reconstruction geometry and topology and finally tuning the digitization, local reco, particle flow code + validation & DQM. So it is a long path

[bsunanda]

@fabiocos @kpedro88 This will not break any workflow currency available

[fabiocos]

+1