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

Pythia8 interface fix which is really working this time #4296

Conversation

tomcornelis
Copy link
Contributor

When the bug in Pythia8 flat QCD weights was found, it was suggested to copy the Pythia8Interface from CMSSW_4_4_7 to fix it (#3853). Even though it compiled nicely, the fix introduced some pythia8 initialization problems, probably due to name changes of functions in GeneratorInterface/Core, I think.

Therefore I made this new fix where I revert Pythia8Hadronizer.cc back to the former version and only updated the sections which deal with the weights. So this is a more minimal fix, as was actually suggested by Josh before. I have tested it with our generator fragments, and it is really working this time.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @UAEDF-tomc (Tom Cornelis) for CMSSW_4_2_X.

Pythia8 interface fix which is really working this time

It involves the following packages:

GeneratorInterface/Pythia8Interface

@vciulli, @nclopezo, @thuer, @cmsbuild, @bendavid, @Degano 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.
@smuzaffar, @nclopezo, @davidlt, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@bendavid
Copy link
Contributor

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_4_2_X IBs unless changes or unless it breaks tests. @smuzaffar, @nclopezo, @davidlt, @ktf can you please take care of it?

@bendavid
Copy link
Contributor

(Every time we build a 42x release a puppy dies keep in mind.)

@tomcornelis
Copy link
Contributor Author

Any idea when the next CMSSW_4_2_10_patch2 will be build?

@vciulli
Copy link
Contributor

vciulli commented Jun 25, 2014

there wasn’t any ORP meeting this week but maybe we can push for it by email

On 25 Jun 2014, at 13:26, Tom Cornelis notifications@github.com wrote:

Any idea when the next CMSSW_4_2_10_patch2 will be build?


Reply to this email directly or view it on GitHub.

@davidlange6
Copy link
Contributor

its already known - but for some reason 710 and 720pre1 have gotten priority…

On Jun 25, 2014, at 2:09 PM, Vitaliano Ciulli notifications@github.com
wrote:

there wasn’t any ORP meeting this week but maybe we can push for it by email

On 25 Jun 2014, at 13:26, Tom Cornelis notifications@github.com wrote:

Any idea when the next CMSSW_4_2_10_patch2 will be build?


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.

davidlange6 added a commit that referenced this pull request Jul 1, 2014
…allyWorkingThisTime

Pythia8 interface fix which is really working this time
@davidlange6 davidlange6 merged commit 680d0a3 into cms-sw:CMSSW_4_2_X Jul 1, 2014
@tomcornelis tomcornelis deleted the pythia8InterfaceFixWhichIsReallyWorkingThisTime branch April 13, 2018 15:43
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