genproductions icon indicating copy to clipboard operation
genproductions copied to clipboard

merge.pl decoupled from MG5

Open agrohsje opened this issue 1 year ago • 1 comments

Dear @menglu21, @bbilin , @colizz, @sihyunjeon ,

I realized a potential problem. In gridpack_generation.sh, we are copying merge.pl to our working path from https://github.com/cms-sw/genproductions/blob/master/bin/MadGraph5_aMCatNLO/Utilities/merge.pl. This code is no longer in sync with what is used by MG5. See e.g. this bug fix by Olivier: https://bazaar.launchpad.net/~maddevelopers/mg5amcnlo/LTS_dev/revision/377 On top, I think we have a python version adapted by @colizz of that code: https://github.com/cms-sw/cmssw/blob/master/GeneratorInterface/LHEInterface/scripts/mergeLHE.py which is also not synced. Questions: Shouldn't we monitor potential changes to this code by the MG5 authors and migrate them to our versions?

I also spotted that we keep a second (not used?) version in genproductions: https://raw.githubusercontent.com/cms-sw/genproductions/master/Utilities/scripts/merge.pl If I am right that it is not used, I would suggest to delete.

Cheers, Alexander.

agrohsje avatar Jan 17 '24 17:01 agrohsje

so in the end this was relevant to the pnr issue! at least we could've been safe from the perl issue if i listened to you carefully :| i think we can migrate to the python version once we test all the corner cases

sihyunjeon avatar Mar 06 '24 07:03 sihyunjeon