Fix Phase II Tracker Material scenarios to remove duplicate 20% one

[mdhildreth]

Fix duplicate 20% percent material to 50 % scenario. D27 was supposed to be 50%

[mdhildreth]

please test

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23814/5508

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/29147/console Started: 2018/07/17 22:56

[cmsbuild]

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

It involves the following packages:

Geometry/CMSCommonData

@civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @kpedro88 can you please review it and eventually sign? Thanks.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

-1

Tested at: d81c8bb

You can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23814/29147/summary.html

I found follow errors while testing this PR

Failed tests: UnitTests

• Unit Tests:

I found errors in the following unit tests:

---> test test2023Geometry had ERRORS

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 2892290
• DQMHistoTests: Total failures: 8
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2892092
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 30 files compared)
• Checked 128 log files, 14 edm output root files, 31 DQM output files

[ianna]

@mdhildreth - you need to update dict2023Geometry.py:
https://cmssdt.cern.ch/lxr/source/Configuration/Geometry/python/dict2023Geometry.py
and then re-generate the scenario to avoid the error:

Running script: /build/cmsbld/jenkins/workspace/ib-any-integration/CMSSW_10_2_X_2018-07-17-1100/src/Configuration/Geometry/test/runtests.sh
cmsExtendedGeometry2023D27XML_cfi.py differs

Please, do not forget to update https://cmssdt.cern.ch/lxr/source/Configuration/Geometry/README.md

[ianna]

-1

[mdhildreth]

please test

[mdhildreth]

@ianna Fixed now. If everything passes, could you please sign?

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23814/5680

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/29399/console Started: 2018/07/24 16:34

[cmsbuild]

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

[mdhildreth]

@ianna p.s. The README file was correct; the dictionary was wrong.

[cmsbuild]

+1
Tested at: 859fc89
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23814/29399/summary.html

[cmsbuild]

Comparison job queued.

[ianna]

+1

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 2889291
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2889100
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 30 files compared)
• Checked 128 log files, 14 edm output root files, 31 DQM output files

[kpedro88]

+upgrade

[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)

[kpedro88]

+1