Skip to content

FindMadGraph Updates, main branch (2024.04.30.)

Attila Krasznahorkay requested to merge akraszna/atlasexternals:FindMadGraphUpdate-main-20240430 into main

Changed how FindMadGraph.cmake would look for MadGraph, following some revelations coming out of AGENE-2244. 🤔

Since the time that the module was originally written, lcg_external_module(...) learned some tricks. Nowadays it can look for packages just containing executables, quite conveniently.

Removed the setting of MADGRAPH_PYTHON_PATH, and replaced it with the setup of MADGRAPH_BINARY_PATH. So that mg5_aMC would end up in our ${PATH}, and so that ${PYTHON_PATH} would not get the MadGraph modules into it. Since as it seems, the latter is not supposed to happen. 😕

@zmarshal, @ewelina, @amdesai, @dhirsch, how should we test this properly? 🤔 Could you give me a recipe that I could try on top of an AthGeneration build to see if these changes make things better?

Merge request reports