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

Problem for CKKW merging algorithm (for the "CMS Drell Yan" process) #645

Closed
oliviermattelaer opened this issue May 10, 2023 · 4 comments · Fixed by #654
Closed

Problem for CKKW merging algorithm (for the "CMS Drell Yan" process) #645

oliviermattelaer opened this issue May 10, 2023 · 4 comments · Fixed by #654
Assignees

Comments

@oliviermattelaer
Copy link
Member

crash in the pure fortran vectorised code

generate p p > ta+ ta-
add process p p > ta+ ta- j
#add process p p > ta+ ta- j j
output PROC_sm_2 --vector_size=2
launch
set ickkw 1
set mc_grouped_subproc F

Findings so far:

  • issue is the behaviour of the findmt function
  • difference with vector_size is the value of iconfig when reaching that routine.
    • When running G2 the vectorised code hase iconfig set to 1 and not to 2
@oliviermattelaer
Copy link
Member Author

Fixed issue: mg5amcnlo/mg5amcnlo@ef334c5
@roiser @valassi

@valassi
Copy link
Member

valassi commented May 16, 2023

Thanks Olivier! For reference, this is relevant to the CMS process suggested by Sapta

They use these cards
https://github.com/cms-sw/genproductions/tree/mg29x/bin/GenValidation/updated_validation_cards/dy_mlm_mll50

And they generate the gridpack with this code:
https://github.com/cms-sw/genproductions/blob/master/bin/MadGraph5_aMCatNLO/gridpack_generation.sh

@roiser
Copy link
Member

roiser commented May 16, 2023

Hi @oliviermattelaer , thanks, was this the issue in "cluster.f" we discussed earlier? cheers

valassi added a commit to valassi/madgraph4gpu that referenced this issue May 17, 2023
valassi added a commit to valassi/madgraph4gpu that referenced this issue May 17, 2023
…for CMS Drell-Yan (madgraph5#645)

NB: Olivier's patch also includes one commit fixing madgraph5#629, making vecsizeFIX unnecessary (at least for the moment)
@valassi valassi changed the title Problem for CKKW merging algorithm Problem for CKKW merging algorithm (for the "CMS Drell Yan" process) May 17, 2023
@valassi
Copy link
Member

valassi commented May 23, 2023

For the record, I confirm that this is fixed in commit mg5amcnlo/mg5amcnlo@6d2182c which I am now using in madgraph4gpu (see MR #654, in 57d8ec3), while I can reproduce an error after launch in mg5amcnlo/mg5amcnlo@c1d10f4

So this is definitely fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants