madgraph5 / madgraph4gpu

GPU development for the Madgraph5_aMC@NLO event generator software package
30 stars 33 forks source link

Updated workplan (May 2023) towards a first alpha release #671

Open valassi opened 1 year ago

valassi commented 1 year ago

This is an updated workplan (as of May 2023) towards a first alpha release. It follows up on and replaces the previous workplan from December 2022 in #576, where several issues have been fixed, but some are still open, and in addition many new ones have appeared.

The plan is essentially that I presented at the dev meeting last Monday in https://indico.cern.ch/event/1288364/

Trying to put things in order (but I also indicate the numbers of #576)

Licensing etc (non technical blocker) - new with respect to #576

General resync of madgraph4gpu to upstream mg5amcnlo (blocker) - this was in point 2 in #576

Relocatable builds - this was part of point 3 in #576, even if not explicitly mentioned there

Handling of multiple P1/P2/... subprocesses (blocker for this common use case) - part of point 6 in #576

Madevent application cleanup (blocker) - this was in point 3 of #576

Cleanup of other build issues (~blocker for c++-only builds?) - new added June 4

Port post-generation patches upstream (blocker) - this was part of 2 in #576

Test the launch functionality (blocker) - this was point 5 in #576

Cross check handling of parameters (blocker) - this was point 4 in #576

Copy plugins to mg5amcnlo (~blocker: the final blocker step?) - new with respect to #576

Cleanup of color (high priority for physics results, but not a technical blocker) - this was point 1 in #576

Experiment integration (our final goal - to be tested both before and after the release) - ~part of point 6 in #576

Other physics validation issues (high priority for the experiments, not strictly technical blockers) - ~part of point 6 in #576

SM process-specific issues (high priority for the experiments, not strictly technical blockers) - ~part of point 6 in #576

BSM process-specific issues (varying, lower priorities) - ~part of point 6 in #576

nscottnichols commented 1 year ago

@valassi I saw your comment here when I was syncing the sycl branch with the latest cudacpp changes. I am getting the correct value for nwf (at least for gg_tt) using self.matrix_elements[0].get_number_of_wavefunctions() in model_handeling.py. It could be because I added code in generate_process_files() to write CPPProcess.cc before CPPProcess.h.

valassi commented 1 year ago

@valassi I saw your comment here when I was syncing the sycl branch with the latest cudacpp changes. I am getting the correct value for nwf (at least for gg_tt) using self.matrix_elements[0].get_number_of_wavefunctions() in model_handeling.py. It could be because I added code in generate_process_files() to write CPPProcess.cc before CPPProcess.h.

Thanks @nscottnichols ! I opened #677 as a reminder that I could also try to move nwf from CPPProcess.cc to CPPProcess.h. This is low priority anyway - so no longer a blocker for the alpha release - because the main issue was moving nwf from src to P1, and this is done. It would be a nice future cleanup/improvement however.

oliviermattelaer commented 2 months ago

I need also to focus on https://github.com/madgraph5/madgraph4gpu/issues/867 to allow the integration of gpucpp within the 3.6.0 branch of MG5aMC