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 to pythia8 209 #1579

Merged
merged 1 commit into from May 27, 2015
Merged

Conversation

mkirsano
Copy link
Contributor

No description provided.

@cmsbuild cmsbuild added this to the Next CMSSW_7_5_X milestone May 26, 2015
@bendavid
Copy link
Contributor

Did the the jet matching patch not need to be updated?

@mkirsano
Copy link
Contributor Author

Matching patch worked as is.

@mkirsano
Copy link
Contributor Author

Question to SW people: what was the reason to introduce the compiler option patch adapted here?

@bendavid
Copy link
Contributor

What's the reason we don't need --with-boost anymore?

@mkirsano
Copy link
Contributor Author

pythia8 uses lhapdf compatibility/access functions. Boost is not used in the compilation.
Actually it turned out that it was so also in the previous version, but the configure script checked for Boost and did not build lhapdf interface in absence of Boost. This is fixed now.

@bendavid
Copy link
Contributor

Ah, ok, I was confused by the release notes, which say
"Linkage to LHAPDF6 so far has been based on the LHAPDF5 compatibility mode, which requires no Boost headers. The new configure option --with-lhapdf6-plugin=LHAPDF6.h uses native mode, and then requires Boost headers."

So we are still using the compatability mode then? Is there any advantage moving to native mode?

@mkirsano
Copy link
Contributor Author

In principle there could be some timing advantages. I don't know about such studies.

@nclopezo
Copy link
Contributor

+1
tested building cmssw-tool-conf

@nclopezo
Copy link
Contributor

@bendavid is this PR ok for you? I would like to confirm before merging it.

@bendavid
Copy link
Contributor

Basic tests are ok (identical results on 1000 minbias events)

@bendavid
Copy link
Contributor

+1

nclopezo added a commit that referenced this pull request May 27, 2015
@nclopezo nclopezo merged commit 6fa3587 into cms-sw:IB/CMSSW_7_5_X/stable May 27, 2015
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.

None yet

4 participants