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

Revision history for this message
marco zaro (marco-zaro) wrote :

Hi Ken,
I cannot see the log (launchpad does not support attachments for code review). Can you send it to me?
In any case, this is rather strange, as no python file has been touched in this branch.

Cheers,

Marco

> On 16 Dec 2020, at 14:12, ken mimasu <email address hidden> wrote:
>
> 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.
>
>
> --
> https://code.launchpad.net/~maddevelopers/mg5amcnlo/3.0.3-betternlops/+merge/394582
> Your team MadDevelopers is subscribed to branch lp:~maddevelopers/mg5amcnlo/3.0.4.

« Back to merge proposal