Skip to content

first version of ee_mumu coming from madgraph --some PR still need to…#69

Merged
oliviermattelaer merged 7 commits intomasterfrom
epoch2
Dec 10, 2020
Merged

first version of ee_mumu coming from madgraph --some PR still need to…#69
oliviermattelaer merged 7 commits intomasterfrom
epoch2

Conversation

@oliviermattelaer
Copy link
Member

Hi,

This is the current output generated by MG5aMC which is relatively close to the current output of epoch1.
I likely miss some recent (less than two weeks old) PR. If you can put those in the comment below, this will help me to check them one by one and to include the associate change.

The mG5aMC code is not yet running for gg>tt~gg but this will likely be working soon.

@roiser
Copy link
Member

roiser commented Nov 27, 2020

Hi @oliviermattelaer there is #61 which needs to be still approved (I'll try to ping reviewers) but includes also some changes for the json output in epoch 1 ee-mumu

@oliviermattelaer
Copy link
Member Author

Thanks Stefan, this should be in now

@oliviermattelaer
Copy link
Member Author

@valassi @roiser @hageboeck
The Cpp code is now compiling (but requires curand obviously so it needs cuda)

@hageboeck should we put the test for epoch2? would be nice to have it setup before merging this branch no?

@hageboeck
Copy link
Member

Yes, let me check out the branch and see if I can get the test to run. You can take the commits once done.

@hageboeck hageboeck mentioned this pull request Dec 1, 2020
3 tasks
@hageboeck
Copy link
Member

hageboeck commented Dec 1, 2020

@oliviermattelaer see #78. I managed to port the tests, and they fail. :-D

PS: You can cherry-pick the makefile and CI-related commits or go on top of #78, i.e. madgraph4gpu/epoch2_tests, to get a github check for epoch2.

@hageboeck
Copy link
Member

@oliviermattelaer I tried to push the commits to activate the CI on top, but I failed. I will reset. :-)

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.

3 participants