Add bin/madevent to the madgraph4gpu repo for all processes#851
Merged
valassi merged 4 commits intomadgraph5:masterfrom May 30, 2024
Merged
Add bin/madevent to the madgraph4gpu repo for all processes#851valassi merged 4 commits intomadgraph5:masterfrom
valassi merged 4 commits intomadgraph5:masterfrom
Conversation
…ubProcesses/P*/.gitignore The main issue here is that this prevents adding bin/madevent to the repo. This script is needed, for instance, to run ./bin/generate_events with a different runcard. Presently this gives this error: ../bin/madevent treatcards run make: ../bin/madevent: No such file or directory This is the complete list of files removed from cudacpp/.gitignore: .libs .cudacpplibs madevent madevent_fortran madevent_cpp madevent_cuda G[0-9]* ajob[0-9]* input_app.txt symfact.dat gensym I will then regenerate all processes including bin/madevent
…g .gitignore - commit (2/2) bin/madevent for dir in $(git ls-tree --name-only HEAD */bin); do git add $dir/madevent; done Note: this only adds four files in BSM processes new file: heft_gg_bb.mad/bin/madevent new file: smeft_gg_tttt.mad/bin/madevent new file: susy_gg_t1t1.mad/bin/madevent new file: susy_gg_tt.mad/bin/madevent
Member
Author
|
PS Ah yes this includes many changes also because it is the first full regeneartion with your PR 835: so launch_plugin.py and runcards are modified from your patch Olivier |
Member
Author
|
PPS and just for further explanation, I removed madevent and a few other files from epochX/cudacpp/.gitignore because it was/is ALREADY in the .gitignore of all SubProcesses/P* generated directories |
valassi
added a commit
to valassi/madgraph4gpu
that referenced
this pull request
May 30, 2024
…h launch PR madgraph5#851 git checkout upstream/master $(git ls-tree --name-only HEAD */CODE*txt)
valassi
added a commit
to valassi/madgraph4gpu
that referenced
this pull request
May 30, 2024
valassi
added a commit
to valassi/madgraph4gpu
that referenced
this pull request
May 30, 2024
…graph5#851 patches - only codelogs change anyway
oliviermattelaer
approved these changes
May 30, 2024
Member
oliviermattelaer
left a comment
oliviermattelaer
left a comment
There was a problem hiding this comment.
Sure go ahead.
Thanks
Olivier
Member
Author
|
Ok thanks! I will merge this. Andrea |
valassi
added a commit
to valassi/madgraph4gpu
that referenced
this pull request
May 30, 2024
…ph5#851 for bin/madevent and .gitignore) into susy
valassi
added a commit
to valassi/madgraph4gpu
that referenced
this pull request
May 30, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi @oliviermattelaer this is a very simple PR that only affecs madgraph4gpu repos. It does not touch CODEGEN.
It adds bin/madevent to the madgraph4gpu repo for all processes, in particular BSM processes where it was missing (while for SM processes it was there). I had to modify the .gitignore (again only of madgraph4gpu, no change in CODEGEN).
The reason is that I need bin/madevent to launch bin/generate_events from a repo process when I modify the runcards. (The treacards step uses it).
I hope it is not controversial, can you approve please? I need this to investigate the cross section crashes in susy #826
Thanks
Andrea