Get latest changes from pythia8 226 on 7_1_X

[mrodozov]

Description: cms-externals/pythia8#13
Resolves #4581

[cmsbuild]

A new Pull Request was created by @mrodozov (Mircho Rodozov) for branch IB/CMSSW_7_1_X/stable.

@cmsbuild, @smuzaffar, @gudrutis, @mrodozov can you please review it and eventually sign? Thanks.
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.

[mrodozov]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32161/console

[cmsbuild]

-1

Tested at: aaa3776

• Build:

I found compilation error when building:

result = commandSpec[0](*commandSpec[1:])
File "PKGTOOLS/cmsBuild", line 3070, in installApt
File "PKGTOOLS/cmsBuild", line 3072, in installApt
File "PKGTOOLS/cmsBuild", line 220, in install_reference
RpmInstallFailed: Failed to install package herwig. Reason:
error: line 1: Tag takes single token only: Name        : external+herwig+6.521-ddibom7  Relocations: /opt/cmssw

* The action "install-external+sip-toolfile+1.0-cms2" was not completed successfully because Traceback (most recent call last):
File "/build/cmsbld/jenkins/workspace/ib-any-integration/PKGTOOLS/scheduler.py", line 199, in doSerial
result = commandSpec[0](*commandSpec[1:])
File "PKGTOOLS/cmsBuild", line 3070, in installApt


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

[mrodozov]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32162/console

[cmsbuild]

-1

Tested at: aaa3776

• Build:

I found compilation error when building:

result = commandSpec[0](*commandSpec[1:])
File "PKGTOOLS/cmsBuild", line 3070, in installApt
File "PKGTOOLS/cmsBuild", line 3072, in installApt
File "PKGTOOLS/cmsBuild", line 220, in install_reference
RpmInstallFailed: Failed to install package herwig. Reason:
error: line 1: Tag takes single token only: Name        : external+herwig+6.521-ddibom7  Relocations: /opt/cmssw

* The action "install-external+sip-toolfile+1.0-cms2" was not completed successfully because Traceback (most recent call last):
File "/build/cmsbld/jenkins/workspace/ib-any-integration/PKGTOOLS/scheduler.py", line 199, in doSerial
result = commandSpec[0](*commandSpec[1:])
File "PKGTOOLS/cmsBuild", line 3070, in installApt


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

[smuzaffar]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32169/console

[cmsbuild]

+1
Tested at: aaa3776
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4582/32169/summary.html

[cmsbuild]

+1
Tested at: aaa3776
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4582/32604/summary.html

[cmsbuild]

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

@slava77 comparisons for the following workflows were not done due to missing matrix map:

• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-4582/503.0_MinBias_13TeV_pythia8+MinBias_13TeV_pythia8+HARVESTGEN
• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-4582/504.0_QCD_Pt-30_13TeV_pythia8+QCD_Pt-30_13TeV_pythia8+HARVESTGEN
• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-4582/505.0_DYToLL_M-50_13TeV_pythia8+DYToLL_M-50_13TeV_pythia8+HARVESTGEN
• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-4582/506.0_WToLNu_13TeV_pythia8+WToLNu_13TeV_pythia8+HARVESTGEN

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 11
• DQMHistoTests: Total histograms compared: 682535
• DQMHistoTests: Total failures: 3349
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 678812
• DQMHistoTests: Total skipped: 374
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: -10 KiB( 10 files compared)
• Checked 44 log files, 4 edm output root files, 11 DQM output files

[alberto-sanchez]

test seems ok.

[alberto-sanchez]

+1

[mseidel42]

I cannot find plots for those GEN workflows but the root files look reasonable 👍

[efeyazgan]

@intrepid42 do you think we can sign it now?

[mseidel42]

Hi Efe, yes, I don't see any Pythia failures, so it is good to go

[Andrej-CMS]

Dear @ALL ,
Are there any further studies required for this PR? It would be very useful to have the PSweights in 2016.
Kind regards,
Andrej

[qliphy]

@smuzaffar We need "externals-approved" here. Can you have a look and sign if possible? We would like to have a patched 71X release for 2016 production with PSweights. Thanks!

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_7_1_X/stable 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]

@qliphy @efeyazgan @alberto-sanchez is this the backport into an older version of a new pythia8 feature not present in the version of 7_1_X?

[qliphy]

@fabiocos Yes, exactly, as you can also see from cms-externals/pythia8#13
this is to bring in 7_1_X the new pythia8 feature of UncertaintyBand options

[fabiocos]

+1