-
Notifications
You must be signed in to change notification settings - Fork 157
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
feat: Add Pythia8 python example + test #1031
feat: Add Pythia8 python example + test #1031
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1031 +/- ##
=======================================
Coverage 48.57% 48.57%
=======================================
Files 338 338
Lines 17369 17369
Branches 8216 8216
=======================================
Hits 8437 8437
Misses 3179 3179
Partials 5753 5753 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. Unfortunately I don't have pythia locally so I cannot test it, @paulgessinger could you run it before merging to confirm it works properly ?
Tested this locally, and the CI runs it as well. |
I didn't think about this... Yeah then it should be fine ! |
This PR adds a python example script demonstrating our Pythia8 integration. It can write out particles to CSV and ROOT formats. Also adds a test for this behavior.