Skip to content

WIP: towards a workflow#620

Closed
roiser wants to merge 22 commits intomadgraph5:masterfrom
roiser:cms-workflow1
Closed

WIP: towards a workflow#620
roiser wants to merge 22 commits intomadgraph5:masterfrom
roiser:cms-workflow1

Conversation

@roiser
Copy link
Member

@roiser roiser commented Mar 31, 2023

This is a tentatively first set of patches towards running the whole workflow. So far this PR contains two sets of patches to

  • fixes for missing includes (symmetry.f) which were for now not compiled
  • create the input files for madevent with the additional parameters for the application execution

more to be added in this PR ...

@valassi
Copy link
Member

valassi commented Jun 3, 2023

Hi @roiser I have cherry picked a few commits from here (related to P1/P2 directories) and am about to merge them in MR #676.

When I am done please rebase at some point

@valassi
Copy link
Member

valassi commented Jun 3, 2023

(By the way the symmetry.f fixes should also no longer be necessary as they are in mg5amcnlo now)

@valassi
Copy link
Member

valassi commented Jun 9, 2023

Hi @roiser I have cherry picked most of your changes (runcard and makefile) in MR #688, where I also rebased this to the latest master.

The only things I have NOT picked up are the additions of new processes in generateAndCompare... maybe I'd keep that for later.

I would tend to close this MR #620 as replaced by #688... but let me know. Thanks

valassi added a commit to valassi/madgraph4gpu that referenced this pull request Jun 12, 2023
valassi added a commit to valassi/madgraph4gpu that referenced this pull request Jun 12, 2023
@valassi
Copy link
Member

valassi commented Jun 12, 2023

Hi Stefan, now ALL of these commits have been included, either in MR #688 or in MR #692. (They have then be modified, or partly undone, but everything was checked and initially included).

I think this can be closed now. Thanks again! Andrea

@valassi valassi closed this Jun 12, 2023
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.

2 participants