Code review comment for lp:~maddevelopers/mg5amcnlo/3.0.3-betternlops

Hi Marco, all,

Apologies, I realised I did not react to this in good time. What is the status of the branch, has it been merged already?
I finally got round to checking it out and trying it with SMEFTatNLO and I have encountered a crash at the output stage that I have never seen before:

> Command "output test_ttbar" interrupted with error:
> Exception : name SMEFTatNLO already consider as a python library cann't be reassigned(/Users/ken/Work/apps/MG5/dev/SMEFTatNLO!=/Users/ken/Work/apps/MG5/dev/test_ttbar/SubProcesses/P0_gg_ttx/V0_gg_ttx/SMEFTatNLO)

I was trying the usual test of generating ttbar but with some EFT operators and there seems to be a python clash of modules somewhere, although when I look in the directory on the right hand side of the inequality it doesn’t contain the module SMEFTatNLO.

I attach the debug log.

Best,

Ken

> On 30 Nov 2020, at 13:55, Ioannis Tsinikos <email address hidden> wrote:
>
> Review: Approve nlo+ps (py8)
>
> Hi Marco,
>
> thanks, I rechecked with your fix and it is ok.
>
> Best,
> Ioannis
> --
> https://eur03.safelinks.protection.outlook.com/?url=https:%2F%2Fcode.launchpad.net%2F~maddevelopers%2Fmg5amcnlo%2F3.0.3-betternlops%2F%2Bmerge%2F394582&amp;data=04%7C01%7Cken.mimasu%40uclouvain.be%7Caaa02d3d8ab24d66412408d89537ab59%7C7ab090d4fa2e4ecfbc7c4127b4d582ec%7C0%7C1%7C637423414175005300%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&amp;sdata=ljNagXeCK0hLR0SC9RKsx1foubOClinlCId4biPni%2Fs%3D&amp;reserved=0
> You are requested to review the proposed merge of lp:~maddevelopers/mg5amcnlo/3.0.3-betternlops into lp:~maddevelopers/mg5amcnlo/3.0.4.