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
multi-thread support for MadGraph LO process (106X) #30030
multi-thread support for MadGraph LO process (106X) #30030
Conversation
A new Pull Request was created by @colizz (Congqiao Li) for CMSSW_10_6_X. It involves the following packages: GeneratorInterface/LHEInterface @SiewYan, @mkirsano, @cmsbuild, @GurpreetSinghChahal, @agrohsje, @alberto-sanchez, @qliphy can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
please test |
The tests are being triggered in jenkins. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
Adding @Dr15Jones @colizz can you give a recipe to test this development with a MadGraph request from RunIISummer19UL17wmLHEGEN? |
Thanks @qliphy. I now added the recipe in the beginning. |
I don't really have any comments as this implementation has no affect on the framework. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_10_6_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_11_2_X is complete. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Implementing multi-thread support for MadGraph LO process.
The
script run_generic_tarball_cvmfs_madgraphLO_multithread.sh
script is designed as a replacement ofrun_generic_tarball_cvmfs.sh
to enable the internal multithreading method of MadGraph. Another file is a patch that works together with this script.More details can be found in the slides.
PR validation:
This PR does not touch the CMSSW framework.
Various gridpacks have been tested with this script for validation.
This PR is a backport from #30029
The implemented script is irrelevant to the CMSSW version, so it can always work.
(A quick test)