Code review comment for lp:~maddevelopers/mg5amcnlo/3.0.1_4FS_merged3.0.2_latest

Revision history for this message
Rikkert Frederix (frederix) wrote :

Hi Marco,

It looks like you have turned on the compute_6to5flav_cnt() stuff (which is now relevant for the 5to4flav). This should be fixed. (Also, these terms should not be added to the Born -> rather, a new 'type' should be created (current list is in add_wgt() subroutine of fks_singular.f). This will probably require a couple of changes throughout the code, but will be the cleanest and most easy to maintain in the future.

Cheers,
Rikkert

review: Needs Fixing

« Back to merge proposal