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

Pythia6 Generator #115

Merged
merged 7 commits into from
Mar 25, 2016
Merged

Pythia6 Generator #115

merged 7 commits into from
Mar 25, 2016

Conversation

nfeege
Copy link
Contributor

@nfeege nfeege commented Mar 18, 2016

Adding Pythia6 as event generator. The code uses the Pythia6 wrapper provided by HepMC to generate events in HepMC format and stores them on the node tree. It uses a configuration file to configure Pythia settings in the same way PHPythia in the PHENIX software does.

The code runs, calls Pythia6, and generates some HepMC output which can be passed through the Geant4 simulation. Crosschecks of the output still need to be done. I just initiated the pull request so everyone who is interested in this can have a look.

This generator will allow us to do Pythia6 / Pyhtia8 crosschecks in the sPHENIX framework and run e+p simulations for EIC.

@mccumbermike
Copy link
Contributor

Jin asked me to review the unit translation from PYTHIA into our output. So I ran a single event and looked for an example of off-vertex decay in the input. Here is the example I found:

screen shot 2016-03-24 at 10 26 33 am

As you can see there is particle 104 generated at vertex -44. The units from pythia are GeV and mm. I then found the same particle in our final output here:

screen shot 2016-03-24 at 10 28 03 am

which is in our canonical GeV and CM units. I see that the units are passed correctly from input to output.

I have one suggestion for the pull request. The smearing interface is still available on the header but is commented out inside. We can smear at the HepMC reader, so I recommend also commenting out the header functions or someone will be confused why they don't work.

@nfeege
Copy link
Contributor Author

nfeege commented Mar 25, 2016

Thanks a lot, Mike!

I renamed PHPythia files, class and library to PHPythia6.

I also got rid of the (non-functional) vertex smearing interface.

@blackcathj blackcathj merged commit 068e16d into sPHENIX-Collaboration:master Mar 25, 2016
@nfeege nfeege deleted the Pythia6 branch March 26, 2016 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants