Skip to content
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

upgrade lhapdf to 6.5.4, sets to 6.5.1c #8852

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

mkirsano
Copy link
Contributor

@mkirsano mkirsano commented Dec 3, 2023

Upgrade lhapdf to 6.5.4, pdf sets to 6.5.1c. The previous version of sets is 6.4.0e. Checked looking at the content that in the new version of sets there are some new sets, the old ones are the symbolic links pointing to 6.4.0e.

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 3, 2023

A new Pull Request was created by @mkirsano (Mikhail Kirsanov) for branch IB/CMSSW_14_0_X/master.

@aandvalenzuela, @iarspider, @smuzaffar, @cmsbuild can you please review it and eventually sign? Thanks.
@rappoccio, @sextonkennedy, @antoniovilela you are the release manager for this.
cms-bot commands are listed here

@iarspider
Copy link
Contributor

iarspider commented Dec 3, 2023 via email

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 3, 2023

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6eccb4/36282/summary.html
COMMIT: 4cb5aa5
CMSSW: CMSSW_14_0_X_2023-12-03-0000/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/8852/36282/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

@smuzaffar
Copy link
Contributor

+externals

@cmsbuild
Copy link
Contributor

cmsbuild commented Dec 4, 2023

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

@iarspider
Copy link
Contributor

iarspider commented Dec 5, 2023

@mkirsano I think this update broke RelVal 132.0 (full log):

----- Begin Fatal Exception 04-Dec-2023 14:45:34 CET-----------------------
An exception of category 'StdException' occurred while
   [0] Processing  Event run: 1 lumi: 1 event: 6 stream: 1
   [1] Running path 'simulation_step'
   [2] Calling method for module Pythia6GeneratorFilter/'generator'
Exception Message:
A std::exception was thrown.
Info file not found for PDF set ''

Could you please check?

@mkirsano
Copy link
Contributor Author

mkirsano commented Dec 6, 2023

I don't imagine how this can be connected. I checked that pythia6 still works with lhapdf and this pdf set. The error occured in event 4 or 5, not immediately.

@stahlleiton
Copy link

stahlleiton commented Dec 19, 2023

A backport for CMSSW 13_0_X and CMSSW_13_0_HeavyIon_X (HIN MC GEN release) and CMSSW 12_4_X (or the latest release meant for Run3 POWHEG and MadGraph5 production)
of this PR (together with #8869 )
is needed to use the latest nuclear PDFs for MC production as include in cms-sw/genproductions#3586

@mkirsano
Copy link
Contributor Author

mkirsano commented Dec 19, 2023 via email

@mkirsano
Copy link
Contributor Author

mkirsano commented Jan 8, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants