Pythia 8.200pre2 (71x)

[bendavid]

Backport of interface changes for pythia 8.2.
Needs to be merged together with
cms-sw/cmsdist#1065

[bendavid]

+1

[cmsbuild]

A new Pull Request was created by @bendavid (Josh Bendavid) for CMSSW_7_1_X.

Pythia 8.200pre2 (71x)

It involves the following packages:

Configuration/Generator
FastSimulation/ParticleDecay
GeneratorInterface/PartonShowerVeto
GeneratorInterface/Pythia8Interface

@cmsbuild, @civanch, @nclopezo, @mdhildreth, @lveldere can you please review it and eventually sign? Thanks.
@ghellwig this is something you requested to watch as well.
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.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[lveldere]

@bendavid

Hi Josh

In #5602 you changed the way the pythia flags are set in this file.
Why not here in 71?

Cheers

Lukas

[bendavid]

Hi Lukas,
In 71x it seemed as though the code setting the flags was not present at all. Was fastsim + pythia 8 really working already in 71x or only in 72x?

[lveldere]

Aha, you're right.

In 71X there was a trial version, implemented probably last year, and not
used.

No reason to bother about it.

Thanks

Lukas

On Mon, Sep 29, 2014 at 12:55 PM, Josh Bendavid notifications@github.com
wrote:

In FastSimulation/ParticleDecay/interface/PythiaDecays.h:

@@ -8,7 +8,7 @@
#include <HepMC/GenEvent.h>
#include <HepMC/GenParticle.h>
#include <Pythia8/Pythia.h>
-#include <Pythia8/Pythia8ToHepMC.h>

Hi Lukas,
In 71x it seemed as though the code setting the flags was not present at
all. Was fastsim + pythia 8 really working already in 71x or only in 72x?

—
Reply to this email directly or view it on GitHub
https://github.com/cms-sw/cmssw/pull/5603/files#r18147404.

[ghost]

+1
Tested against cms-sw/cmsdist#1065.
Usual matrix tests ran fine: 401.0,1306.0,101.0,1003.0,50101.0,9.0.

[lveldere]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes or unless it breaks tests. @Degano can you please take care of it?

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes or unless it breaks tests. @Degano can you please take care of it?