Code review comment for lp:~mg5core2/mg5amcnlo/2.5.5

Revision history for this message
Olivier Mattelaer (olivier-mattelaer) wrote :

Hi Rik,

> Hasn't this always been the case?

That was my first reaction when I realised that they were missing. So I checked that point explicitly and indeed this is something new (since 2.4.3).

> I thought that the only really important information to safe was the relative contributions of all the channels, so that one exactly knows how many events each channel should give.

I agree that this is the crucial information. Now discarding the grid is wasting a lot of information.
I also have to double check this but in the last run I did, it was written that the grid was kept fixed.
If this is the case, starting from the default grid sounds a bad idea.

Cheers,

Olivier

> On 20 May 2017, at 02:47, Rikkert Frederix <email address hidden> wrote:
>
> Hi Olivier,
>
> Hasn't this always been the case? I thought that the gridpacks became too large when all the grids are saved. I thought that the only really important information to safe was the relative contributions of all the channels, so that one exactly knows how many events each channel should give.
>
> Best,
> Rikkert
>
> --
> https://code.launchpad.net/~mg5core2/mg5amcnlo/2.5.5/+merge/324347
> Your team mg5core2 is subscribed to branch lp:~mg5core2/mg5amcnlo/2.5.5.

« Back to merge proposal