New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tunesetforpythia8vgt235 #26802
Tunesetforpythia8vgt235 #26802
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-26802/9806
|
A new Pull Request was created by @efeyazgan (Efe Yazgan) for master. It involves the following packages: Configuration/Generator @alberto-sanchez, @cmsbuild, @qliphy, @agrohsje, @efeyazgan can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
-1 Tested at: 872e332 You can see the results of the tests here: I found follow errors while testing this PR Failed tests: AddOn
I found errors in the following addon tests: cmsRun /cvmfs/cms-ib.cern.ch/week0/slc7_amd64_gcc700/cms/cmssw-patch/CMSSW_11_0_X_2019-05-15-2300/src/PhysicsTools/PatAlgos/test/IntegrationTest_cfg.py : FAILED - time: date Thu May 16 17:10:32 2019-date Thu May 16 17:05:22 2019 s - exit: 23552 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+1 |
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) |
@efeyazgan is this update desired for the UL production? If yes this would impact the production of the GEN-SIM PU library, am I correct? |
@efeyazgan as far as I can see, for the pdf this is just a renaming but the actual pdf does not change, am I correct? |
@efeyazgan @pgunnell ok, but this means that 10_6_0 cannot be used to produce the library, it needs this update at minimum |
@fabiocos the access method changed from pythia8 to lhapdf method (pythia8 authors decided to get rid of our main NNLO pdf from pythia8 internal sets). But this one we put as default is the positive-definite version of our pdf set which allows us to use this pdf up to arbitrarily high scales w/o the problem of negative-weights. In the standard model phase space this pdf set and our previous default are equivalent. |
But we added these settings buy hand in the minbias production with 10_6. |
@efeyazgan @pgunnell so this means that the settings are passed manually so far, and we will get this in 10_6_1? Please provide a backport for this |
+1 |
Updating tunes to account for changes in pythia8 version >= 2.3.5
Please see slide 7 of https://indico.cern.ch/event/810088/sessions/307594/attachments/1824497/2990874/pcgt_GEN_Apr_10_2019.pdf